On Mon, Feb 07, 2005 at 07:38:55PM +0100, Erwan MAS wrote:
> Package: zile
> Severity: important
> Tags: patch
> 
> I have a crash of zile , on AMD64 platform .
> The patch in attachement solve this problem .

  Can you detail how zile crashed ?
  I compiled and used zile on amd64 and did not manage to make it crash
in 2 minutes.

> The message :
> | Zile crashed.  Please send a bug report to <[EMAIL PROTECTED]>.
> | Trying to save modified buffers (if any)...
> 
> The patch has been integrated in upstream .
> 
> http://cvs.sourceforge.net/viewcvs.py/zile/zile/src/minibuf.c?r1=1.27&r2=1.28

   I agree that the line your patch delete is not needed. But I
don't understand how it can prevent a crash.

   If the vasprintf succeed, the vsprintf call must succeed also since
the buffer pointed by buf has been allocated by vasprintf. By definition
of vasprintf, buf is large enough for the vsprintf call.
   So as I understand that code the two lines are redundant but does
not introduce a bug.

   Is that correct ?

-- 
Nicolas Duboc <[EMAIL PROTECTED]>

Attachment: signature.asc
Description: Digital signature

Reply via email to