Hello again,

On Thu, Jun 02, 2005 at 07:25:41PM +0200, Martin Schulze wrote:
> Florian Ernst wrote:
> > I believe this change wasn't aimed at preventing an integer overflow
> > at all, but rather at preventing a "malloc(0)".
> [...]
> --> superflous crap for a security update.

Agreed.

> > This is my interpretation of this change, please hit me (hard) with a
> > cluebat if I'm wrong.
> 
> I had hoped you would have come up with a different explanation, since
> this is what I thought as well. :-%

No, sorry.


On Thu, Jun 02, 2005 at 07:57:06PM +0200, Martin Schulze wrote:
> I've looked at the patch you've provided and I must say that I believe
> that it is utterly broken with regards to the "integer overflow".  I
> don't think that I've discovered a single integer overflow that's
> been prevented.   Attached is what was left over after the investigation.
> [...]
> Please correct me if I'm wrong.
> Please run a diff agains the interdiff between the stable
> package and the "fixed" packages.

Done, your attachment looks OK to me. I'm now quite unsure about the
wording of the changelog entry, though.

Thanks a lot for your investigation,
cheers,
Flo

Attachment: signature.asc
Description: Digital signature

Reply via email to