[Pmk-devel] pmksetup fix
Brought to you by:
coudercd
|
From: Damien C. <mip...@us...> - 2005-01-28 13:42:24
|
Hi all ! Due to lack of testers the 0.9.1 release has been provided with two bugs. The first is that using pmksetup with -v argument is flushing the configuration file content. The second is leaving the temporary file of pmksetup if ran without correct privileges. The attached diff is splitting the arguments parsing in two parts : the first part is now in main() and process -v and -V flags that are safe while in privileged mode. The other allowed arguments are parsed in the child (non privileged) part to reduce risks. The result is that -v is exiting really early now. The renaming/saving part of the configuration file is now done only if child status is okay. Another change is that the temporary file is also written in the same directory than the configuration file. This directory is also checked and created earlier than before to avoid running tests if the results cannot be written. If you have some time to review and test this diff i'd be happy to get some feedback to quickly release a bugfix release. Cheers, Damien |