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]