Donate Share

Logging Framework for C++

Tracker: Patches

5 Bugfix for PatternLayout - ID: 1661157
Last Update: Comment added ( rahlf )

Hi,

when building log4cpp-0.3.5rc3 "make check" did not succeed on my
Linux/Debian box (gcc version 4.1.2 20061115 (prerelease) (Debian
4.1.1-21)).

I found a bug in PatternLayout.cpp where an (unsigned) size_t variable is
checked "> 0". This is obviously always true and an IndexError is raised
occasionally.

BTW: My g++ complained about an "extra ';'".

bye, matthias


Matthias Rahlf ( rahlf ) - 2007-02-16 04:53

5

Closed

Invalid

Vaclav Haisman

None

None

Public


Comments ( 3 )

Date: 2008-09-01 22:50
Sender: rahlf


You are right of course. Shame on me!
I was evaluating logging libraries and it seems I got confused and mistook
you for your evil twin brother :-)


Date: 2008-09-01 11:48
Sender: wilxProject Admin


I think you have confused log4cpp and log4cplus.


Date: 2007-02-20 23:32
Sender: rahlf


ok, not "obviously always" (== 0).
But whenever the preceeding subtraction is negative, it leads to an
IndexError.


Attached File ( 1 )

Filename Description Download
unsigned.patch Download

Changes ( 5 )

Field Old Value Date By
status_id Open 2008-09-01 11:48 wilx
resolution_id None 2008-09-01 11:48 wilx
assigned_to nobody 2008-09-01 11:48 wilx
close_date - 2008-09-01 11:48 wilx
File Added 216251: unsigned.patch 2007-02-16 04:53 rahlf