Organization: Red Hat UK Ltd. Registered Address: Red Hat UK Ltd, Amberley
	Place, 107-111 Peascod Street, Windsor, Berkshire, SI4 1TE, United
	Kingdom.
	Registered in England and Wales under Company Registration No. 3798903
From: David Howells <dhowells@redhat.com>
In-Reply-To: <28281.1439431881@warthog.procyon.org.uk>
References: <28281.1439431881@warthog.procyon.org.uk> <alpine.LRH.2.20.1508131058380.7538@namei.org> <10418.1439399192@warthog.procyon.org.uk>
Cc: dhowells@redhat.com, James Morris <jmorris@namei.org>, mcgrof@gmail.com,
        zohar@linux.vnet.ibm.com, dwmw2@infradead.org,
        linux-kernel@vger.kernel.org, linux-security-module@vger.kernel.org
Subject: Re: [GIT PULL] MODSIGN: Use PKCS#7 for module signatures [ver #8]
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 8BIT
Date: Thu, 13 Aug 2015 03:55:46 +0100
Message-ID: <28480.1439434546@warthog.procyon.org.uk>
To: unlisted-recipients:; (no To-header on input)
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1075
Lines: 28

David Howells <dhowells@redhat.com> wrote:

> > I'm still seeing these warnings:
> > 
> > scripts/sign-file.c: In function ‘main’:
> > scripts/sign-file.c:188: warning: value computed is not used
> 
> Ummm...  What do you see on line 188?  "BIO_reset(b);"?  If so, that seems to
> be an openssl bug.  b is created four lines above and definitely used on the
> following line, so the problem must lie with the BIO_reset() function or
> macro.

It seems that the definition of BIO_reset() is the same in 1.0.1e as in
1.0.1k.

I was assuming that the value computed that is not being used is 'b' or some
internal state, but I can't see how that is the case.  I think it's actually
the result of the function - which is being cast, otherwise your compiler
would just ignore it.

I presume you're using the RHEL-6 compiler?

David
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/