On Tue, 2012-10-09 at 18:31 -0400, Yaroslav Halchenko wrote: > ok -- let's then just wait for Dave's feedback -- it might be that this > all came up purely from code introspection and indeed not pertinent per > se. > > On Tue, 09 Oct 2012, Albert Chu wrote: > > > > * debian/patches/0002_excel_when_opening_tmp.patch: Open files with > > O_EXCL. > > > Sorry, I think I got confused when I first read this and the code. > > > Now that I'm looking through the code, the debug data is dumped > > to /var/log/ipmiconsole. It's only when FreeIPMI is being compiled in > > debug/developer mode that files are temporarily stored in /tmp. So we > > shouldn't consider the storage of files into /tmp the normal > > production/release case. > > > So I'm even more confused now on the need for O_EXCL. The production > > case is exactly what you'd want. Files stored into /var/log/ipmiconsole > > and already existing is ok. > > besides two ipmiconsoles trying to write into the same file, right > -- or do you expect that work just fine?
This may not work fine. Lets wait for Dave's response. See what he originally intended and then move forward. There could be bugs, but they may be different than what the patch proposes. -- Albert Chu ch...@llnl.gov Computer Scientist High Performance Systems Division Lawrence Livermore National Laboratory -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
