diff options
| author | Richard Hughes <richard@hughsie.com> | 2007-12-23 23:01:45 +0000 |
|---|---|---|
| committer | Richard Hughes <richard@hughsie.com> | 2007-12-23 23:01:45 +0000 |
| commit | aac703ae48779eab946664035c9e7aa2d079b43d (patch) | |
| tree | c431cccfd05b300ddedc6255092c2d272edb514a | |
| parent | ca7e573f6eecf69148cb20d1868b0a72dfc6a78c (diff) | |
2007-12-23 Richard Hughes <richard@hughsie.com>
* pm/functions:
There is no need to remove the logfile, because the '>' after exec in
the next line already truncates the logfile to zero length.
Also removing is wrong, because it destroys the selinux context of the
logfile. Bugzilla Reports:
https://bugzilla.redhat.com/show_bug.cgi?id=237840
https://bugzilla.redhat.com/show_bug.cgi?id=238068
| -rw-r--r-- | ChangeLog | 10 | ||||
| -rwxr-xr-x | pm/functions | 1 |
2 files changed, 10 insertions, 1 deletions
@@ -1,6 +1,16 @@ 2007-12-23 Richard Hughes <richard@hughsie.com> * pm/functions: + There is no need to remove the logfile, because the ">" after exec in + the next line already truncates the logfile to zero length. + Also removing is wrong, because it destroys the selinux context of the + logfile. Bugzilla Reports: + https://bugzilla.redhat.com/show_bug.cgi?id=237840 + https://bugzilla.redhat.com/show_bug.cgi?id=238068 + +2007-12-23 Richard Hughes <richard@hughsie.com> + + * pm/functions: - Check for the individual config file, not the list. 2007-09-10 Peter Jones <pjones@redhat.com> diff --git a/pm/functions b/pm/functions index 62d7761..b370137 100755 --- a/pm/functions +++ b/pm/functions @@ -137,7 +137,6 @@ do_suspend_hybrid() pm_main() { if [ -n "$PM_LOGFILE" ]; then - [ -f "$PM_LOGFILE" ] && rm -f "$PM_LOGFILE" exec > "$PM_LOGFILE" 2>&1 fi take_suspend_lock || exit 1 |