[Bug 409631] New: [Novell:NTS/ntsutils] schealth does not check BHFILE permissions
https://bugzilla.novell.com/show_bug.cgi?id=409631 Summary: [Novell:NTS/ntsutils] schealth does not check BHFILE permissions Product: openSUSE.org Version: unspecified Platform: Other OS/Version: Other Status: NEW Severity: Normal Priority: P5 - None Component: 3rd party software AssignedTo: jrecord@novell.com ReportedBy: jrecord@novell.com QAContact: opensuse-communityscreening@forge.provo.novell.com Found By: Development If a basic-health-check.txt file is copied with it's default root.root and 600 mode, and then schealth is run as a normal user, there are permission errors reading the file. schealth should check the file mode first. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=409631 Jason Record <jrecord@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=409631 User jrecord@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=409631#c1 Jason Record <jrecord@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |RESOLVED Resolution| |FIXED --- Comment #1 from Jason Record <jrecord@novell.com> 2008-07-16 15:39:01 MDT --- Fix checked in. Index: trunk/scripts/schealth =================================================================== --- trunk/scripts/schealth (revision 27) +++ trunk/scripts/schealth (working copy) @@ -1,5 +1,5 @@ #!/bin/bash -SVER=0.95.1 +SVER=0.95.2 ############################################################################## # schealth - Supportconfig Health Check Report @@ -160,15 +160,22 @@ pout Health Check Files reset_lstatus BHFILE="$(pwd)/${BHFILE}" - if [ -s $BHFILE ]; then + if [ -s $BHFILE -a -r $BHFILE ]; then set_status green - else + if (( OPT_VERBOSE )); then + print_lstatus Err "Using: $BHFILE" + else + print_lstatus + fi + elif [ ! -s $BHFILE ]; then set_status error - fi - if (( OPT_VERBOSE )); then - print_lstatus "File not found, $BHFILE" "Using: $BHFILE" + print_lstatus "File not found, $BHFILE" + elif [ ! -r $BHFILE ]; then + set_status error + print_lstatus "Read permission denied, $BHFILE" else - print_lstatus "File not found, $BHFILE" + set_status warning + print_lstatus "Unknown error, $BHFILE" fi test $LSTATUS -gt 0 && { footer; abort_log; } } -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=409631 User jrecord@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=409631#c2 Jason Record <jrecord@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED --- Comment #2 from Jason Record <jrecord@novell.com> 2008-08-01 09:33:04 MDT --- Closed. Released to OBS http://en.opensuse.org/NTSutils -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
participants (1)
-
bugzilla_noreply＠novell.com