Re: i386/mm and openwall change

From: Maciej Zenczykowski
Date: Tue Jan 20 2004 - 18:08:40 EST


Perhaps, however I assumed that errorcode == 0 is a special case meaning
no error, this seems to be suggested by the code. If so then the ow
approach differs from the current 25pre6 code. The openwall sets it to 0
meaning (I assume) no error, while the 25pre6 code sets it to a non-zero
value, while the 24 code sets it to whatever errorcode was (can it be
zero or nonzero? probably both, since this would explain the change
in the first place...). This effectively means we have 3 different
function source-codes for the address>=TASK_SIZE case, while all behave
identically for address<TASK_SIZE.

On Tue, 20 Jan 2004, Linus Torvalds wrote:
> On Tue, 20 Jan 2004, Maciej Zenczykowski wrote:
> > I'm assuming this means the openwall change was an error?
>
> Oh, the openwall approach is fine too - the exact error number doesn't
> really matter much, and which one you choose is pretty much a matter of
> taste.
> Linus

If it doesn't matter why set the bit at all? If we do set the least
significant bit this is obviously because it shouldn't be zero (i.e. it
doesn't matter as long as it's boolean true), if so then why does ow set
it to zero in this case. Obviously this is somehow _weird_...

Cheers,
MaZe.


-
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/