Hello Folks,
 
   Follow attach a patch which addressed the issue reported on BZ#836984.

   Thank you.

Best Regards,
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:33:55 -0200
Subject: [PATCH] 836984 - fixes the permissions on /var/log/osad log file

---
 client/tools/osad/osad.spec      | 7 +++++++
 client/tools/osad/src/rhn_log.py | 7 +++----
 2 files changed, 10 insertions(+), 4 deletions(-)

diff --git a/client/tools/osad/osad.spec b/client/tools/osad/osad.spec
index 57014b7..eaa0a8e 100644
--- a/client/tools/osad/osad.spec
+++ b/client/tools/osad/osad.spec
@@ -276,6 +276,13 @@ rpm -ql osa-dispatcher | xargs -n 1 /sbin/restorecon -rvi {}
 /sbin/restorecon -vvi /var/log/rhn/osa-dispatcher.log
 %endif
 
+# Fix the /var/log/osad permission BZ 836984
+%post
+if [ -f %{_var}/log/osad ]; then
+    /bin/chmod 600 %{_var}/log/osad
+fi
+%endif
+
 %files
 %dir %{rhnroot}/osad
 %attr(755,root,root) %{_sbindir}/osad
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

Reply via email to