Opened 9 years ago

Closed 9 years ago

#6123 closed defect (fixed)

Can we change the interface to OGRAPISpyPostClose?

Reported by: Kurt Schwehr Owned by: warmerdam
Priority: normal Milestone:
Component: default Version: svn-trunk
Severity: normal Keywords:
Cc:

Description

Why is OGRAPISpyPostClose getting passed an invalid pointer? Can we drop the pointer arg since it is invalid? Or can we at least pass a nullptr? I'd rather to that than make a comment to suppress the Coverity complaint. 

    if( bOGRAPISpyEnabled )
   	
    CID 138126 (#1 of 1): Use after free (USE_AFTER_FREE)
    5. pass_freed_arg: Passing freed pointer hDS as an argument to OGRAPISpyPostClose.
    73        OGRAPISpyPostClose(hDS);

Change History (2)

comment:1 by Even Rouault, 9 years ago

This is an internal API. You can remove the unused argument

comment:2 by Even Rouault, 9 years ago

Resolution: → fixed
Status: new → closed

trunk r30714 "Remove unused argument of OGRAPISpyPostClose() (#6123)"

Note: See TracTickets for help on using tickets.