[Freeipmi-devel] [bug #14993] libfreeipmi sensor event messages mem-leaks (ipmi-sensor-event-messages.c)

[Anand Babu]

Update of bug #14993 (project freeipmi):

                  Status:                    None => Fixed                  
             Assigned to:                    None => balamurugan            
             Open/Closed:                    Open => Closed                 

    _______________________________________________________

Follow-up Comment #1:

All the internal functions which strdup in ipmi-sensor-event-messages.c are
declared static now. 2 high level APIs around them compose the allocated
dynamic strings into a list of strings (char **) and return them. User (fish:
scm-procedures.c) is already freeing them properly in scm-procedures.c.

So there are no known memory leaks in that file.

Bala and me had a lengthy discussion on approaches other than strdup, like
static string arrays, user-allocated buffers ..
But came to conclusion that the current approach is good.

--
Anand Babu



    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/bugs/?func=detailitem&item_id=14993>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.gnu.org/