This message generated a parse failure. Raw output follows here. Please use 'back' to navigate. From devnull@lkml.org Thu Apr 25 11:53:19 2024 Delivery-date: Sat, 28 Jan 2006 19:52:51 +0100 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S932551AbWA1Swd (ORCPT ); Sat, 28 Jan 2006 13:52:33 -0500 Received: from zproxy.gmail.com ([64.233.162.203]:27657 "EHLO zproxy.gmail.com") by vger.kernel.org with ESMTP id S932547AbWA1Swc convert rfc822-to-8bit (ORCPT ); Sat, 28 Jan 2006 13:52:32 -0500 Received: by zproxy.gmail.com with SMTP id 34so796981nzf for ; Sat, 28 Jan 2006 10:52:31 -0800 (PST) DomainKey-Signature: a=rsa-sha1; q=dns; c=nofws; s=beta; d=gmail.com; h=received:message-id:date:from:to:subject:cc:in-reply-to:mime-version:content-type:content-transfer-encoding:content-disposition:references; b=nJjeNXAHu7Njn6HR9wCa65N9Yp17nktYJnOTRFJ9 Received: by 10.65.133.15 with SMTP id k15mr1146403qbn; Sat, 28 Jan 2006 10:52:31 -0800 (PST) Received: by 10.65.114.19 with HTTP; Sat, 28 Jan 2006 10:52:31 -0800 (PST) Message-Id: <9a8748490601281052h56fe8dc9r511b450c46e353d1@mail.gmail.com> Date: Sat, 28 Jan 2006 19:52:31 +0100 From: Jesper Juhl To: Andrew Morton Subject: Re: [PATCH] ide-tape: attempt to handle copy_*_user() failures [take two] Cc: linux-kernel@vger.kernel.org, axboe@suse.de In-Reply-To: <20060123000401.144e2ca2.akpm@osdl.org> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7BIT Content-Disposition: inline References: <200601222251.56185.jesper.juhl@gmail.com> <20060123000401.144e2ca2.akpm@osdl.org> Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org On 1/23/06, Andrew Morton wrote: > Jesper Juhl wrote: > > > > Second attempt at this patch. > > > > ( > > noticed a small flaw in my first patch, this should be better. > > also noticed that Gadi Oxman seems unable to recieve mail at the address > > listed in MAINTAINERS, so put a few other (hopefully relevant) people on > > Cc instead > > ) > > > > > > Attempt to handle copy_*_user() failures in > > drivers/ide/ide-tape.c::idetape_copy_stage_*_user > > > > drivers/ide/ide-tape.c:2663: warning: ignoring return value of opy_from_user', declared with attribute warn_unused_result > > drivers/ide/ide-tape.c:2690: warning: ignoring return value of opy_to_user', declared with attribute warn_unused_result > > > > Due to lack of hardware I'm unfortunately not able to test this patch > > beyond making sure it compiles. > > > > Too scary for me. > Fair enough. > > @@ -2669,26 +2671,30 @@ static void idetape_copy_stage_from_user > > if (bh) > > atomic_set(&bh->b_count, 0); > > } > > + if (not_copied) > > + return not_copied; > > } > > tape->bh = bh; > > + return 0; > > } > > What are the effects of not updating tape->bh? > Ouch, I guess the effects would be the next access to the tape hitting the same area as the last one - not good. I don't know this code very intimately and this certainly looks like a mistake on my part, so ignore the patch for now please. I made the patch fully aware that I was not on very solid ground, which is also why I didn't ask for "merge this please" and used the "Attempt to handle .." language in the patch description - I was just offering the patch for "here you are, see if it looks good" review. The review turned up flaws, so for now the patch is dead (thank you for looking at it though). -- Jesper Juhl Don't top-post http://www.catb.org/~esr/jargon/html/T/top-post.html Plain text mails only, please http://www.expita.com/nomime.html - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/