Edit dive info, cancel, crash

Dirk Hohndel dirk at hohndel.org
Sun Jul 6 11:52:59 PDT 2014


On Sun, Jul 06, 2014 at 08:28:51AM -0700, Dirk Hohndel wrote:
> Valgrind shows me an issue with this sequence.
> Specifically, we try to copy the samples of our dive as we start to edit
> it (loadFromDive()) and then find out that those samples had been freed
> the last time we started editing that dive.
> 
> I'm trying to track this down...

And a few minutes of looking at the code, adding one printf and spending
ten minutes trying to figure out why it was never called... bingo.

I did not copy the events / samples for the first DC (the one that's part
of the struct dive).

Just pushed the fix.

/D


More information about the subsurface mailing list