New Ticket     Tickets     Wiki     Browse Source     Timeline     Roadmap     Ticket Reports     Search

Ticket #17772 (closed defect: fixed)

Opened 4 years ago

Last modified 4 years ago

rrdtool 1.3.5: missing #include <libgen.h> for basename() in rrd_graph.c

Reported by: vinc17@… Owned by: markd@…
Priority: Normal Milestone:
Component: ports Version: 1.7.0
Keywords: build Cc:
Port: rrdtool

Description

I've reported the following bug to upstream: http://oss.oetiker.ch/rrdtool-trac/ticket/202

With RRDtool 1.3.5 under Mac OS X at least, one needs a

#include <libgen.h>

line in rrd_graph.c, as the prototype of the basename() function (used in this C source) is defined in this header. Otherwise there's a type mismatch (the compiler emits warnings), which may have incorrect side effects on some platforms (e.g., if the pointers do not have the same size as an int).

See attached patch.

Attachments

include.diff (834 bytes) - added by vinc17@… 4 years ago.

Change History

Changed 4 years ago by vinc17@…

comment:1 Changed 4 years ago by vinc17@…

  • Status changed from new to closed
  • Resolution set to fixed

Fixed in 1.3.6.

comment:2 Changed 4 years ago by anonymous

  • Milestone Port Bugs deleted

Milestone Port Bugs deleted

Note: See TracTickets for help on using tickets.