On Sat, Jun 03, 2006 at 01:41:51PM +0200, Steinar H. Gunderson wrote: > On Sun, May 21, 2006 at 09:40:04AM -0700, Matt Kraai wrote: > > I believe that Doxygen is overflowing the data buffer when it calls > > sprintf. The attached patch makes it allocate enough memory before > > calling sprintf, which should fix the problem. > > Shouldn't QCString::sprintf() be fixed instead, using vsnprintf and checking > the return value? That seems like a much more reasonable fix than just > slapping bigger buffers in everywhere. > > I'll write up a patch for that and NMU -- it looks like the most sane > solution.
I created a patch to do that originally, but I wasn't sure that upstream would want to add a dependency on vnsprintf. -- Matt
signature.asc
Description: Digital signature
