[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#344481: Mysterious crash in php4-rrdtool



On Fri, Dec 23, 2005 at 02:44:21AM +0100, Artur R. Czechowski wrote:
> On Thu, Dec 22, 2005 at 04:45:25PM -0800, Steve Langasek wrote:
> > On Fri, Dec 23, 2005 at 01:09:31AM +0100, Artur R. Czechowski wrote:
> > > The rrd_graph_options() gets a string with parameters. The begin if this
> > > string is (at least in my case, but this is common value in rrdtools):
> > > --start -2d
> > > The rrd_graph_options calls getopt_long to split parameters into tokens.
> > > You can see the code in rrdtool's sources, file rrd_graph.c, line 2965.
> > > getopt_long reckognises the --start parameter correctly and, according to
> > > struct option returns opt as s. But, for some strange reasons, it sets
> > > optarg for NULL.
> > Erm, don't you need to write --start=-2d if you want an optarg that begins
> > with a hyphen?
> No. The "s:" in optstring and 1 as a second value in {"start",1,0,'s'} always
> makes next token a parameter.

Ok.

> > Anyway, I don't see where any of the other packages you've mentioned are
> > providing a broken implementation of getopt_long.  Shouldn't you check where
> > this "broken" function is coming from, and file the bug there?
> Wish I know how to do it. Simple grep in sources gives no results.
> That's why it is so mysterious to me and I am asking for help at wider
> auditorium.

Break on rrd_graph_options, step into getopt_long, run bt to see what lib it
says it's in.

> Because only apache/php4(module) is a problematic combination I suspect
> an apache. If it is sufficient for you I have no objections against
> reassigning this bug to apache package.

I'd rather see some analysis to show where the bug actually belongs instead
of guessing and reassigning.

-- 
Steve Langasek                   Give me a lever long enough and a Free OS
Debian Developer                   to set it on, and I can move the world.
vorlon@debian.org                                   http://www.debian.org/

Attachment: signature.asc
Description: Digital signature


Reply to: