On 12/06/2012 03:42 PM, Marcelo Moreira de Mello wrote:
Hello Folks,
Here follow a better looking patch. Please, ignore the previous one.
Thank you.
Best,
mmello
--
Marcelo Moreira de Mello
RHCA RHCSS RHCVA
Senior Software Maintenance Engineer/SEG
gpg id: 2048R/FDB110E5
gpg fingerprint: 3BE7 EF71 4DD7 6812 D309 8F18 BD42 D095 FDB1 10E5
From: Marcelo Moreira de Mello <mme...@redhat.com>
Date: Thu, 6 Dec 2012 15:58:03 -0200
Subject: [PATCH] 836984 - fixes the permissions on /var/log/osad log file
---
client/tools/osad/osad.spec | 5 +++++
client/tools/osad/src/rhn_log.py | 7 +++----
2 files changed, 8 insertions(+), 4 deletions(-)
diff --git a/client/tools/osad/osad.spec b/client/tools/osad/osad.spec
index 57014b7..b954b59 100644
--- a/client/tools/osad/osad.spec
+++ b/client/tools/osad/osad.spec
@@ -202,6 +202,11 @@ rm -rf $RPM_BUILD_ROOT
if [ -f %{_sysconfdir}/init.d/osad ]; then
/sbin/chkconfig --add osad
fi
+
+# Fix the /var/log/osad permission BZ 836984
+if [ -f %{_var}/log/osad ]; then
+ /bin/chmod 600 %{_var}/log/osad
+fi
%endif
%preun
diff --git a/client/tools/osad/src/rhn_log.py b/client/tools/osad/src/rhn_log.py
index 7b23501..0b1f85d 100644
--- a/client/tools/osad/src/rhn_log.py
+++ b/client/tools/osad/src/rhn_log.py
@@ -39,10 +39,9 @@ class Logger:
if not Logger.logfile is None:
try:
- file = open( Logger.logfile, 'a' )
- os.chmod(Logger.logfile, 0600)
- file.write( outstring )
- file.close()
+ fd = os.open(Logger.logfile, os.O_APPEND | os.O_RDWR | os.O_CREAT, 0600)
+ os.write(fd, outstring)
+ os.close()
except IOError:
raise
--
1.8.0.1
_______________________________________________
Spacewalk-devel mailing list
Spacewalk-devel@redhat.com
https://www.redhat.com/mailman/listinfo/spacewalk-devel
