From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from metis.ext.pengutronix.de ([2001:67c:670:201:290:27ff:fe1d:cc33]) by bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux)) id 1cmF39-0004oU-VC for barebox@lists.infradead.org; Fri, 10 Mar 2017 07:33:22 +0000 Date: Fri, 10 Mar 2017 08:32:58 +0100 From: Sascha Hauer Message-ID: <20170310073258.nahg4i2wmcrni5s2@pengutronix.de> References: <20170308092402.21245-1-o.rempel@pengutronix.de> <20170308092402.21245-2-o.rempel@pengutronix.de> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20170308092402.21245-2-o.rempel@pengutronix.de> List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "barebox" Errors-To: barebox-bounces+u.kleine-koenig=pengutronix.de@lists.infradead.org Subject: Re: [PATCH 2/2] usb: fastboot: umount device before writing to it To: Oleksij Rempel Cc: barebox@lists.infradead.org On Wed, Mar 08, 2017 at 10:24:02AM +0100, Oleksij Rempel wrote: > From: Sascha Hauer > > When a fastboot target refers to a device and we are going to write > to it, umount it beforehand, so that we do not end up with corrupt > data after writing. > > Signed-off-by: Sascha Hauer > --- > drivers/usb/gadget/f_fastboot.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/usb/gadget/f_fastboot.c b/drivers/usb/gadget/f_fastboot.c > index a6192b9eb..8d69aea8b 100644 > --- a/drivers/usb/gadget/f_fastboot.c > +++ b/drivers/usb/gadget/f_fastboot.c > @@ -754,6 +754,7 @@ static void cb_flash(struct usb_ep *ep, struct usb_request *req, const char *cmd > goto out; > } > > + umount(filename); I'm not sure about this one. We should probably rather bail out with an error when it's mounted. When it's mounted then the result is undesired, even when we unmount it before flashing. Sascha -- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | _______________________________________________ barebox mailing list barebox@lists.infradead.org http://lists.infradead.org/mailman/listinfo/barebox