Changes between Initial Version and Version 1 of Ticket #9528, comment 9


Ignore:
Timestamp:
May 20, 2014, 10:24:41 AM (10 years ago)
Author:
ttcoder

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #9528, comment 9

    initial v1  
    2525I'm now going to comment out the free() (i.e. leak the memory), see if it "fixes" the driver. Failing that I'll try to outright comment out the 'delete volume', though that would represent a bigger memory leak..
    2626
     27**
    2728
     29EDIT: what's weird is that I've also tried adding the following:
     30
     31{{{
     32    free(fName);
     33+fName = (char*)0xdead3003;
     34}
     35}}}
     36
     37yet that changes absolutely nothing, still get a KDL page fault on 0x7a7a7a7a .. I suppose it means either
     381. the fName pointer is copied.. But I see that mistake nowhere in the cdda code (and I didn't find evidence either of class Volume being copied via its bitwise copy constructor, which is also a common coding mistake)..
     392. cdda is actually an innocent bystander, and somebody else is messing with its legitimately owned fName memory buffer?
     403. something else?