[prev in list] [next in list] [prev in thread] [next in thread] 

List:       busybox
Subject:    Re: [PATCH] rx: Fix file corruption on block checksum failure
From:       Denys Vlasenko <vda.linux () googlemail ! com>
Date:       2011-11-23 12:41:28
Message-ID: CAK1hOcOc1aYk6Z9PBy5ppbKFn_v0m3HH63bcSLFeUfsV=4Jt_A () mail ! gmail ! com
[Download RAW message or body]

On Tue, Nov 22, 2011 at 9:29 PM, Dan Fandrich <dan@coneharvesters.com> wrote:
> On Fri, Nov 04, 2011 at 01:58:42PM -0700, Daniel Fandrich wrote:
>> Rather than dropping the bad block, rx was appending it
>> to the file.
>>
>> Signed-off-by: Daniel Fandrich <dan@coneharvesters.com>
>> ---
>>  miscutils/rx.c |    1 +
>>  1 files changed, 1 insertions(+), 0 deletions(-)
>>
>> diff --git a/miscutils/rx.c b/miscutils/rx.c
>> index c48a61f..af59732 100644
>> --- a/miscutils/rx.c
>> +++ b/miscutils/rx.c
>> @@ -207,6 +207,7 @@ static int receive(/*int read_fd, */int file_fd)
>>               continue;
>>   error:
>>   timeout:
>> +             blockLength = 0;
>>               errors++;
>>               if (errors == MAXERRORS) {
>>                       /* Abort */
>
> This patch was never applied. It looks like a duplicate of the one that was
> commited the day before, but it's a related but new issue that I didn't
> notice before.

Applied, thanks!
-- 
vda
_______________________________________________
busybox mailing list
busybox@busybox.net
http://lists.busybox.net/mailman/listinfo/busybox
[prev in list] [next in list] [prev in thread] [next in thread]