All of lore.kernel.org
 help / color / mirror / Atom feed
* Re: Question about the function,ipmi_release in ipmi_devinf.c
       [not found] <54974BAA.9020107@gmail.com>
@ 2014-12-23 14:13 ` Corey Minyard
  0 siblings, 0 replies; only message in thread
From: Corey Minyard @ 2014-12-23 14:13 UTC (permalink / raw)
  To: nick; +Cc: openipmi-developer, linux-kernel

On 12/21/2014 04:37 PM, nick wrote:
> Greetings Again Corney,
> Hope you are looking forward to the holidays. I am one question, about the function in my message.
> /* FIXME - free the messages in the list. */, is in this function I am wondering is this needed?
> Further more the only reason I can see for it is if we are to free the linked list of files or 
> inodes passed to the function, otherwise it's invalid. Please let me known if should sent in 
> a patch to remove it or add a for_each_through linked list call that deletes each file or inode
> in the list passed.

The recv_msgs list needs to be cleaned up in this case.  Basically, just
call ipmi_free_recv_msg()
on every message in the list.  A patch would be appreciated.

Thanks,

-corey


^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2014-12-23 14:13 UTC | newest]

Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <54974BAA.9020107@gmail.com>
2014-12-23 14:13 ` Question about the function,ipmi_release in ipmi_devinf.c Corey Minyard

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.