From: jsing <jsing@openbsd.org>
Date: Fri, 11 Oct 2024 12:10:12 +0000 (+0000)
Subject: Make ERR_str_{libraries,reasons,functs}[] const.
X-Git-Url: http://artulab.com/gitweb/?a=commitdiff_plain;h=955724bc1e7a3eb645e4c5e02daa0c629ab4391b;p=openbsd

Make ERR_str_{libraries,reasons,functs}[] const.

Provide err_load_const_strings(), which takes a const ERR_STRING_DATA *
and does not perform a library error value fixup. Make ERR_str_*[] tables
const.

ok tb@
---

diff --git a/lib/libcrypto/err/err.c b/lib/libcrypto/err/err.c
index 4342efbdab8..d3900a75f46 100644
--- a/lib/libcrypto/err/err.c
+++ b/lib/libcrypto/err/err.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: err.c,v 1.66 2024/10/11 11:58:53 jsing Exp $ */
+/* $OpenBSD: err.c,v 1.67 2024/10/11 12:10:12 jsing Exp $ */
 /* Copyright (C) 1995-1998 Eric Young (eay@cryptsoft.com)
  * All rights reserved.
  *
@@ -141,7 +141,7 @@ typedef struct err_state_st {
 } ERR_STATE;
 
 #ifndef OPENSSL_NO_ERR
-static ERR_STRING_DATA ERR_str_libraries[] = {
+static const ERR_STRING_DATA ERR_str_libraries[] = {
 	{ERR_PACK(ERR_LIB_NONE,0,0),		"unknown library"},
 	{ERR_PACK(ERR_LIB_SYS,0,0),		"system library"},
 	{ERR_PACK(ERR_LIB_BN,0,0),		"bignum routines"},
@@ -174,7 +174,7 @@ static ERR_STRING_DATA ERR_str_libraries[] = {
 	{0, NULL},
 };
 
-static ERR_STRING_DATA ERR_str_functs[] = {
+static const ERR_STRING_DATA ERR_str_functs[] = {
 	{ERR_PACK(ERR_LIB_SYS,SYS_F_FOPEN, 0),     	"fopen"},
 	{ERR_PACK(ERR_LIB_SYS,SYS_F_CONNECT, 0),	"connect"},
 	{ERR_PACK(ERR_LIB_SYS,SYS_F_GETSERVBYNAME, 0),	"getservbyname"},
@@ -188,7 +188,7 @@ static ERR_STRING_DATA ERR_str_functs[] = {
 	{0, NULL},
 };
 
-static ERR_STRING_DATA ERR_str_reasons[] = {
+static const ERR_STRING_DATA ERR_str_reasons[] = {
 	{ERR_R_SYS_LIB,				"system lib"},
 	{ERR_R_BN_LIB,				"BN lib"},
 	{ERR_R_RSA_LIB,				"RSA lib"},
@@ -609,7 +609,7 @@ ERR_get_state(void)
 static void
 err_load_strings(int lib, ERR_STRING_DATA *str)
 {
-	while (str->error) {
+	while (str->error != 0) {
 		if (lib)
 			str->error |= ERR_PACK(lib, 0, 0);
 		err_set_item(str);
@@ -617,6 +617,15 @@ err_load_strings(int lib, ERR_STRING_DATA *str)
 	}
 }
 
+static void
+err_load_const_strings(const ERR_STRING_DATA *str)
+{
+	while (str->error != 0) {
+		err_set_item(str);
+		str++;
+	}
+}
+
 static unsigned long
 get_error_values(int inc, int top, const char **file, int *line,
     const char **data, int *flags)
@@ -688,9 +697,9 @@ ERR_load_ERR_strings_internal(void)
 {
 	err_init_thread = pthread_self();
 #ifndef OPENSSL_NO_ERR
-	err_load_strings(0, ERR_str_libraries);
-	err_load_strings(0, ERR_str_reasons);
-	err_load_strings(0, ERR_str_functs);
+	err_load_const_strings(ERR_str_libraries);
+	err_load_const_strings(ERR_str_reasons);
+	err_load_const_strings(ERR_str_functs);
 	build_SYS_str_reasons();
 	err_load_strings(ERR_LIB_SYS, SYS_str_reasons);
 #endif
@@ -723,10 +732,7 @@ void
 ERR_load_const_strings(const ERR_STRING_DATA *str)
 {
 	ERR_load_ERR_strings();
-	while (str->error) {
-		err_set_item(str);
-		str++;
-	}
+	err_load_const_strings(str);
 }
 
 void