CERTI - Bugs: bug #24294, crash after...

Group

bug #24294: crash after MessageBuffer::read/write_...s() with zero length

* Mandatory Fields

Add a New Comment Rich Markup

Thu 18 Sep 2008 02:43:43 PM UTC, comment #2: Fixed.	Petr Gotthard <gotthardp>
Thu 18 Sep 2008 10:28:55 AM UTC, comment #1: I think yes we should: data[0] may be replaced by: data==NULL ? 0 : data[0] concerning performance you are right, but in debug the first data is insightful :=) I think that debugging output should be suppressed alltogether when not compiling in debug mode. Current "PrettyPrinter" class is not made like this but we may change this in the future in order to have better performance for CERTI compiled for "Release".	Eric NOULARD <erk>
Thu 18 Sep 2008 09:21:23 AM UTC, original submission: The read/write_...s functions have a debug output that print first element in the field that has been saved/loaded (data[0]). If the list is empty (len=0) this may cause segmentation fault. Should we add a condition (that affects the performance) or stop printing first element?	Petr Gotthard <gotthardp>

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:


Comment:

No files currently attached

Depends on the following items: None found

Items that depend on this one: None found

Carbon-Copy List

-email is unavailable- added by erk (Posted a comment)

-email is unavailable- added by gotthardp (Submitted the item)

Please enter the title of George Orwell's famous dystopian book (it's a date):

Follow 4 latest changes.

Date	Changed by	Updated Field	Previous Value	Replaced by
2008-09-21	erk	Open/Closed	Open	Closed
		Fixed Release	Unknown	3.3.1
2008-09-18	gotthardp	Status	None	Ready For Test
		Assigned to	None	gotthardp