On Sat, Jun 03, 2006 at 07:14:12PM +0200, Steinar H. Gunderson wrote:
> There's a bug in the patch I sent; it should be ">= size()", not "> size".
> That still doesn't help, though, so I'm doing a valgrind run now with
> debugging doxygen. I guess we'll see...
AHA!
The bug isn't in QCString at al
On Sat, Jun 03, 2006 at 09:42:00AM -0700, Matt Kraai wrote:
> I created a patch to do that originally, but I wasn't sure that
> upstream would want to add a dependency on vnsprintf.
Well, this is a ticking bomb for more bugs -- just thinking "hey, we'll just
assume the input is below X bytes" is g
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, wh
I tried implementing it, and ended up with the attached patch. However, it
doesn't fix the llvm bug, so I'll have to trace further -- it might be a
different problem.
diff -u doxygen-1.4.6/debian/changelog doxygen-1.4.6/debian/changelog
--- doxygen-1.4.6/debian/changelog
+++ doxygen-1.4.6/debian/ch
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, u
tag 357722 patch
thanks
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.
--
Matt
diff -ru doxygen-1.4.6-orig/src/util.cpp doxygen-1.4.6/src/util.cpp
--- doxygen-1
6 matches
Mail list logo
