Omit sha512_block_data_order() prototype when assembly is not being used.
authorjsing <jsing@openbsd.org>
Tue, 11 Apr 2023 10:26:29 +0000 (10:26 +0000)
committerjsing <jsing@openbsd.org>
Tue, 11 Apr 2023 10:26:29 +0000 (10:26 +0000)
In the case that the pure C implementation of SHA512 is being used, the
prototype is unnecessary as the function is declared static and exists
in dependency order. Simply omit the prototype rather than using #ifndef
to toggle the static prefix.

ok tb@

lib/libcrypto/sha/sha512.c

index d79e287..3fe20de 100644 (file)
@@ -1,4 +1,4 @@
-/* $OpenBSD: sha512.c,v 1.25 2023/04/11 10:21:02 jsing Exp $ */
+/* $OpenBSD: sha512.c,v 1.26 2023/04/11 10:26:29 jsing Exp $ */
 /* ====================================================================
  * Copyright (c) 1998-2011 The OpenSSL Project.  All rights reserved.
  *
 #define SHA512_BLOCK_CAN_MANAGE_UNALIGNED_DATA
 #endif
 
-#ifndef SHA512_ASM
-static
+#ifdef SHA512_ASM
+void sha512_block_data_order(SHA512_CTX *ctx, const void *in, size_t num);
 #endif
-void sha512_block_data_order (SHA512_CTX *ctx, const void *in, size_t num);
 
 #ifndef SHA512_ASM
 static const SHA_LONG64 K512[80] = {