Comment 19 for bug 308060

Revision history for this message
Jamie Strandboge (jdstrand) wrote :

Aurélien, can you clarify your position? Are you saying no patch is needed at all or that your patch is fine as is? The main point Kees was making was to make sure that we don't have a negative length, where you put that check doesn't really matter so long as it ensures that the length is > 0.