Drizzle

Potential barriers bug

Bug #736871 reported by Olaf van der Spek
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Drizzle
Confirmed
Low
Brian Aker

Bug Description

It's about drizzled\session\property_map.h
In setProperty an existing property might be overwritten, in which case the old value isn't freed.
If you add "assert(not _properties.count(arg));" test "execute.concurrent" fails.

Lee Bieber (kalebral-deactivatedaccount)
Changed in drizzle:
status: New → Confirmed
importance: Undecided → Low
assignee: nobody → Brian Aker (brianaker)
Revision history for this message
Brian Aker (brianaker) wrote :

Did you fix this while refactoring that code?

Changed in drizzle:
status: Confirmed → Incomplete
Revision history for this message
Olaf van der Spek (olafvdspek) wrote :

No, I didn't. How should it be fixed?
Should the previous entry be deleted or should this situation not occur in the first place?

Revision history for this message
Olaf van der Spek (olafvdspek) wrote :

And why didn't valgrind pick up the memory leak?

Olaf van der Spek (olafvdspek)
Changed in drizzle:
status: Incomplete → New
status: New → Confirmed
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.