Re: Possible format string vulnerability
Philip Tellis <[email protected]> Mon, 24 Aug 2015 12:57:37 -0400
| Newsgroups | gmane.network.instant-messaging.ayttm.user |
|---|---|
| Message-ID | <CAJ4Gv_aB+1rrO7wO+tOQQ-LFVW2bhw9T+o_Er6kCZ-RG2jngzw@mail.gmail.com> |
--===============6900499558339819720== Content-Type: multipart/alternative; boundary=047d7beb98aaf59ba3051e1184e2 --047d7beb98aaf59ba3051e1184e2 Content-Type: text/plain; charset=UTF-8 Ok, I've looked through the code, and these are my findings: 1. The code should have been inside an #ifdef block, but for some reason it is inside a #ifndef block, which inverts the meaning of the condition. 2. The snprintf and debug_print statements should really just be merged into a single statement with no actual need for debug_buff Unfortunately this code dates back to the very first revision, so it probably inherited from "everybuddy", but I don't have the time to go into the cvs history for EB. I've fixed this and hope someone can test it thoroughly. Philip --047d7beb98aaf59ba3051e1184e2 Content-Type: text/html; charset=UTF-8 Content-Transfer-Encoding: quoted-printable <div dir=3D"ltr">Ok, I've looked through the code, and these are my fin= dings:<div><br></div><div>1. The code should have been inside an #ifdef blo= ck, but for some reason it is inside a #ifndef block, which inverts the mea= ning of the condition.</div><div><br></div><div>2. The snprintf and debug_p= rint statements should really just be merged into a single statement with n= o actual need for debug_buff</div><div><br></div><div>Unfortunately this co= de dates back to the very first revision, so it probably inherited from &qu= ot;everybuddy", but I don't have the time to go into the cvs histo= ry for EB.</div><div><br></div><div>I've fixed this and hope someone ca= n test it thoroughly.</div><div><br></div><div>Philip</div><div><br></div><= div class=3D"gmail_extra"><div class=3D"gmail_quote"><br></div> </div></div> --047d7beb98aaf59ba3051e1184e2-- --===============6900499558339819720== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline ------------------------------------------------------------------------------ --===============6900499558339819720== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ Ayttm-users mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/ayttm-users --===============6900499558339819720==--