Menu

#1154 Unnecessary checks for NULL

pending-accepted
sfcb (1090)
2
2011-09-14
2008-02-12
Tim Potter
No

This patch fixes some instances where a mallocated variable is checked for NULL before freeing it, but the pointer has already been dereferenced previously so the check is unnecessary.

Discussion

  • Tim Potter

    Tim Potter - 2008-02-12
     
  • Chris Buccella

    Chris Buccella - 2008-02-13
    • labels: --> sfcb
    • assigned_to: nobody --> buccella
     
  • Chris Buccella

    Chris Buccella - 2008-02-16
    • priority: 5 --> 2
    • status: open --> open-later
     
  • Chris Buccella

    Chris Buccella - 2011-07-04
    • status: open-later --> open-accepted
     
  • Narasimha Sharoff

    Although it is good to remove unwanted checks, until allocation related check are managed correctly, it is better to leave it as is.

     
  • Narasimha Sharoff

    • assigned_to: buccella --> nsharoff
    • status: open-accepted --> pending-accepted
     

Log in to post a comment.