Hello,

On Wed, Apr 16, 2008 at 10:21:13PM +0200, Nico Golde wrote:
> >     if(path[0]!='/')
> > -           sprintf(tmp,"%s/translations/%s",DATAPATH,path);
> > +           snprintf(tmp,302,"%s/translations/%s",DATAPATH,path);
> 
> off-by two. Why don't you just use sizeof(tmp)?
> Kind regards
> Nico
---end quoted text---

  Actually for this one, tmp is declared as: char tmp[302];

  I will use sizeof(tmp) anyways.

  So is this patch enough to close the bug ?

-- 
 أحمد المحمودي (Ahmed El-Mahmoudy)
  Digital design engineer
  SySDSoft, Inc.
 GPG KeyID: 0x9DCA0B27 (@ subkeys.pgp.net)
 GPG Fingerprint: 087D 3767 8CAC 65B1 8F6C  156E D325 C3C8 9DCA 0B27



-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Reply via email to