Re: [PATCH]staging: slicoss: Fix possible reuse of freed memory in timer function

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Denis,

I understand your frustration; it¹s not that we don¹t want to help, it¹s
that we¹ve moved onto other products.  As for getting H/W to people, I can
easily ship H/W within the United States; it¹s difficult for us to ship
internationally due to many different customs restrictions.  If there is
somewhere I can ship H/W in the states, I¹ll gladly do so.

Best regards,

Chris

On 2/11/14, 2:37 AM, "Denis Kirjanov" <kirjanov@xxxxxxxxx> wrote:

>On 2/10/14, Kirill Tkhai <tkhai@xxxxxxxxx> wrote:
>> Do not call kfree() till timer function is finished.
>>
>> [This was found using grep. Compiled tested only]
>>
>> Signed-off-by: Kirill Tkhai <tkhai@xxxxxxxxx>
>> CC: Joe Perches <joe@xxxxxxxxxxx>
>> CC: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>
>
>CC Chris Harrer.
>
>The problem with that driver is that alacritech guys don't seem to
>care. Not sure if anyone has a hardware to test the driver and
>probably the code is completely broken.
>Probably the better option is to drop the code from the staging tree...
>
>> ---
>>  drivers/staging/slicoss/slicoss.c | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>> diff --git a/drivers/staging/slicoss/slicoss.c
>> b/drivers/staging/slicoss/slicoss.c
>> index 1426ca4..e0de497 100644
>> --- a/drivers/staging/slicoss/slicoss.c
>> +++ b/drivers/staging/slicoss/slicoss.c
>> @@ -2970,7 +2970,7 @@ static void slic_card_cleanup(struct sliccard
>>*card)
>>  {
>>  	if (card->loadtimerset) {
>>  		card->loadtimerset = 0;
>> -		del_timer(&card->loadtimer);
>> +		del_timer_sync(&card->loadtimer);
>>  	}
>>
>>  	slic_debug_card_destroy(card);
>> _______________________________________________
>> devel mailing list
>> devel@xxxxxxxxxxxxxxxxxxxxxx
>> http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
>>
>
>
>-- 
>Regards,
>Denis


_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel





[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux