This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
[PING][PATCH][BZ #16877] Fix typo in nscd/selinux.c
- From: OndÅej BÃlka <neleai at seznam dot cz>
- To: libc-alpha at sourceware dot org
- Date: Sat, 3 May 2014 11:18:54 +0200
- Subject: [PING][PATCH][BZ #16877] Fix typo in nscd/selinux.c
- Authentication-results: sourceware.org; auth=none
- References: <20140427181904 dot GA31159 at domone dot podge>
On Sun, Apr 27, 2014 at 08:19:04PM +0200, OndÅej BÃlka wrote:
>
> This bug is another report of typo in code. A perm variable
> is unitialized so there could be spurious error message
> depending how gcc optimizes this. This could be changed into
> intended check as below.
>
> OK to commit this?
>
> [BZ #16877]
> * nscd/selinux.c (nscd_request_avc_has_perm): Check if there is
> nscd security class.
>
> diff --git a/nscd/selinux.c b/nscd/selinux.c
> index 9a8a5a8..eaed6dd 100644
> --- a/nscd/selinux.c
> +++ b/nscd/selinux.c
> @@ -372,7 +372,7 @@ nscd_request_avc_has_perm (int fd, request_type req)
> /* Get the security class for nscd. If this fails we will likely be
> unable to do anything unless avc_deny_unknown is 0. */
> sc_nscd = string_to_security_class ("nscd");
> - if (perm == 0 && avc_deny_unknown == 1)
> + if (sc_nscd == 0 && avc_deny_unknown == 1)
> dbg_log (_("Error getting security class for nscd."));
>
> /* Convert permission to AVC bits. */
--
magnetic interference from money/credit cards