All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v2 1/4] sign-file: stop referring to the code as main.c
@ 2018-10-22 10:44 James Bottomley
  2018-10-22 11:34 ` David Woodhouse
  0 siblings, 1 reply; 2+ messages in thread
From: James Bottomley @ 2018-10-22 10:44 UTC (permalink / raw)
  To: keyrings

All errors in sign-file print out a line number in main.c.  Fix this
by using the proper __FILE__ designation.

Signed-off-by: James Bottomley <James.Bottomley@HansenPartnership.com>
---
 scripts/sign-file.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/scripts/sign-file.c b/scripts/sign-file.c
index fbd34b8e8f57..563419bc2837 100644
--- a/scripts/sign-file.c
+++ b/scripts/sign-file.c
@@ -76,7 +76,7 @@ void format(void)
 	exit(2);
 }
 
-static void display_openssl_errors(int l)
+static void display_openssl_errors(const char *f, int l)
 {
 	const char *file;
 	char buf[120];
@@ -84,7 +84,7 @@ static void display_openssl_errors(int l)
 
 	if (ERR_peek_error() = 0)
 		return;
-	fprintf(stderr, "At main.c:%d:\n", l);
+	fprintf(stderr, "At %s:%d:\n", f, l);
 
 	while ((e = ERR_get_error_line(&file, &line))) {
 		ERR_error_string(e, buf);
@@ -105,7 +105,7 @@ static void drain_openssl_errors(void)
 #define ERR(cond, fmt, ...)				\
 	do {						\
 		bool __cond = (cond);			\
-		display_openssl_errors(__LINE__);	\
+		display_openssl_errors(__FILE__, __LINE__);\
 		if (__cond) {				\
 			err(1, fmt, ## __VA_ARGS__);	\
 		}					\
@@ -299,7 +299,7 @@ int main(int argc, char **argv)
 
 		/* Digest the module data. */
 		OpenSSL_add_all_digests();
-		display_openssl_errors(__LINE__);
+		display_openssl_errors(__FILE__, __LINE__);
 		digest_algo = EVP_get_digestbyname(hash_algo);
 		ERR(!digest_algo, "EVP_get_digestbyname");
 
-- 
2.16.4

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH v2 1/4] sign-file: stop referring to the code as main.c
  2018-10-22 10:44 [PATCH v2 1/4] sign-file: stop referring to the code as main.c James Bottomley
@ 2018-10-22 11:34 ` David Woodhouse
  0 siblings, 0 replies; 2+ messages in thread
From: David Woodhouse @ 2018-10-22 11:34 UTC (permalink / raw)
  To: keyrings

[-- Attachment #1: Type: text/plain, Size: 307 bytes --]

On Mon, 2018-10-22 at 11:44 +0100, James Bottomley wrote:
> All errors in sign-file print out a line number in main.c.  Fix this
> by using the proper __FILE__ designation.
> 
> Signed-off-by: James Bottomley <James.Bottomley@HansenPartnership.com
> >

Acked-by: David Woodhouse <dwmw@amazon.co.uk>

[-- Attachment #2: smime.p7s --]
[-- Type: application/x-pkcs7-signature, Size: 5213 bytes --]

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2018-10-22 11:34 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-10-22 10:44 [PATCH v2 1/4] sign-file: stop referring to the code as main.c James Bottomley
2018-10-22 11:34 ` David Woodhouse

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.