[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug-diffutils] bug#26155: Test brief-vs-stat-zero-kernel-lies fails if
From: |
Jim Meyering |
Subject: |
[bug-diffutils] bug#26155: Test brief-vs-stat-zero-kernel-lies fails if /proc/cmdline exists but is not readable |
Date: |
Wed, 22 Mar 2017 21:43:16 -0700 |
On Sat, Mar 18, 2017 at 7:56 AM, <address@hidden> wrote:
> Package: diffutils
> Version: 3.5
>
> The test tests/brief-vs-stat-zero-kernel-lies fails with
>
> cat: /proc/cmdline: Permission denied
>
> if /proc/cmdline exists but is not readable. This is for example the case on
> a Linux kernel with grsecurity patchset and option GRKERNSEC_PROC_USER
> enabled.
>
> The fix is easy. The script tests/brief-vs-stat-zero-kernel-lies checks in
> line 13 only for exists (test -f) of $boot (= /proc/cmdline):
>
> test -f $boot || skip_ no $boot file
>
> making it check for readability (test -r) solves the problem (as the test is
> now correctly skipped):
>
> test -r $boot || skip_ file $boot not readable
Thank you for the report and fix.
I propose to push this commit in your name. Since all I have for your
"real name" is "julama", that's what I've used in the commit "author:"
header. If you'd prefer something else, or a different email address,
let me know.
diff-vs-unreadable-proc-cmdline.diff
Description: Text document