[Stgt-devel] [PATCH] Use correct return value of lseek

[Albert Pauw]

FUJITA Tomonori wrote:
> On Fri, 18 Jul 2008 18:37:24 +0200
> Albert Pauw <albert.pauw at gmail.com> wrote:
>
>   
>> Just a small patch.
>>
>> The wrong return value was checked in the if statement.
>>
>> Albert
>>
>> --------
>>
>> --- bs_ssc.c	2008-07-17 19:19:10.000000000 +0200
>> +++ bs_ssc.c.new	2008-07-18 18:34:15.000000000 +0200
>> @@ -65,7 +65,7 @@
>>  	case REZERO_UNIT:
>>  		rew = lseek(fd, 0, SEEK_SET);
>>  		curr_pos = lseek(fd, 0, SEEK_CUR);
>> -		if (ret)
>> +		if (rew)
>>  			set_medium_error(&result, &key, &asc);
>>  		eprintf("Rewind Successful, File Pointer at %" PRIu64",%m\n",
>>  			curr_pos);
>>     
>
> I think that the fix is right but what's for the second lseek? Can we
> remove it?
>
>   
I believe the second seek is put in to be used only by the eprintf 
function, just to check the
current position and show that the seek was succesfull.

Since it is still work in progress and anything near finished I would 
leave it in for the moment.

Albert