Re: [RFC] LZO de/compression support - take 3

From: Nitin Gupta
Date: Wed May 23 2007 - 10:21:55 EST


On 5/23/07, Michael-Luke Jones <mlj28@xxxxxxxxx> wrote:
On 23 May 2007, at 15:03, Nitin Gupta wrote:

>> Perhaps a rename is in order:
>> lzo1x_decompress() => lzo1x_decompress_unsafe()
>> lzo1x_decompress_safe => lzo1x_decompress()
>
> Or perhaps make reiserfs use _safe() instead - I think Richard has
> already submitted patch for same.

If someone's already made this mistake once, then it'll happen again.
In-kernel memory corruption is no fun.

Safe/Secure == Default

If I rename 'nonsafe' version as such then it will seem like its a
'broken' implementation which is not the case. If somebody is upto
including compression he must be having head to use the right
decompress version depending on this scenario :-)

- Nitin




Michael-Luke


-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/