#2 Fix for bug 801857

closed
None
5
2003-11-04
2003-09-07
No

This would be one possible fix for bug 801857. The
malloc()'d string is later free()'d like normal.

Discussion

  • Michael Schwendt

    Logged In: YES
    user_id=62241

    Additionally, free_db_line(..) in db.c does

    checked_free(dl->filename);
    checked_free(dl->linkname);

    two times. It does no harm, however. But you could get rid
    of the comment at the bottom of the function where both
    pointers are check the 2nd time:

    /*
    Possible bug.
    Maybe we shouldn't free filename and linkname...
    */
    checked_free(dl->filename);
    checked_free(dl->linkname);

     
  • Michael Schwendt

    revised patch against db_disk.c

     
  • Richard van den Berg

    Logged In: YES
    user_id=330646

    Applied patch and suggestion to CVS.

     
  • Richard van den Berg

    • assigned_to: nobody --> rvdb
    • status: open --> closed
     

Log in to post a comment.