Menu

#214 fru unreachable

version-cvs
open
nobody
None
5
2013-03-14
2013-03-14
No

become dead code with fixed leaks

Discussion

  • Ales Ledvinka

    Ales Ledvinka - 2013-03-14
    • milestone: --> version-cvs
     
  • Ales Ledvinka

    Ales Ledvinka - 2013-03-14
    • summary: fru originally insufficient now unreachable --> fru unreachable
     
  • Ales Ledvinka

    Ales Ledvinka - 2013-03-14

    the details message of the ticket is probably not accurate. the fixed leaks are at different parts so I guess the dead code was simply skipped at that time.

     
  • Ales Ledvinka

    Ales Ledvinka - 2013-03-14

    hmm just reached the place with leaks again
    lib/ipmi_fru.c:1218: dead_error_line: Execution cannot reach this statement "free(fru_data);".

    lib/ipmi_fru.c:1220: leaked_storage: Variable "fru_data" going out of scope leaks the storage it points to.

     
  • Ales Ledvinka

    Ales Ledvinka - 2013-03-14

    cool. new leaks other then the pending old tracker.

     
  • Ales Ledvinka

    Ales Ledvinka - 2013-03-14
     
  • Ales Ledvinka

    Ales Ledvinka - 2013-03-14

    the new patch hopefully does the right thing. converts the dead and leaking code to what it was probably supposed to do originally.

     

Log in to post a comment.

MongoDB Logo MongoDB