linux-kernel - Re: Question about the function,ipmi_release in ipmi

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [day] [month] [year] [list]

Date:	Tue, 23 Dec 2014 08:13:09 -0600
From:	Corey Minyard <minyard@....org>
To:	nick <xerofoify@...il.com>
CC:	openipmi-developer@...ts.sourceforge.net,
	linux-kernel@...r.kernel.org
Subject: Re: Question about the function,ipmi_release in ipmi_devinf.c

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

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/