]> git.ipfire.org Git - thirdparty/openssh-portable.git/commitdiff
fix previous; check for C99 compound literals
authorDamien Miller <djm@mindrot.org>
Mon, 9 Sep 2024 06:46:40 +0000 (16:46 +1000)
committerDamien Miller <djm@mindrot.org>
Mon, 9 Sep 2024 06:46:40 +0000 (16:46 +1000)
The previous commit was incorrect (or at least insufficient), the
ML-KEM code is actually using compound literals, so test for them.

configure.ac
defines.h

index 9b8b1ded361ab89fc47b6e341db3bca4a225cdd4..954de6cafac1766f5804d97e1d969315e4f76550 100644 (file)
@@ -353,16 +353,17 @@ AC_COMPILE_IFELSE(
     [ AC_MSG_RESULT([no]) ]
 )
 
-AC_MSG_CHECKING([if compiler supports named struct initialisers])
+AC_MSG_CHECKING([if compiler supports compound literals])
 AC_COMPILE_IFELSE(
     [AC_LANG_PROGRAM([[#include <stdlib.h>]],
-    [[ struct foo { int bar; int baz; };
-       struct foo blerg = {.bar = 1, .baz = 2};
-       exit((blerg.bar == 1 && blerg.baz == 2) ? 0 : 1);
+    [[  struct foo { int bar; int baz; };
+        void *fooc = &(struct foo){.bar = 1, .baz = 2};
+        struct foo *foo2 = (struct foo *)fooc;
+        return (foo2->bar == 1 && foo2->baz == 2) ? 0 : 1;
     ]])],
     [ AC_MSG_RESULT([yes])
-      AC_DEFINE(NAMED_STRUCT_INITIALISERS, [1],
-        [compiler supports named struct initializers]) ],
+      AC_DEFINE(COMPOUND_LITERALS, [1],
+        [compiler supports compound literals]) ],
     [ AC_MSG_RESULT([no]) ]
 )
 
index d1f6ce33e974ae5a9fc4c37b33f0991cfaf65172..3d03b033a3c869e55ba715422c8c87ae063c5a79 100644 (file)
--- a/defines.h
+++ b/defines.h
@@ -942,8 +942,8 @@ struct winsize {
 #if defined(VARIABLE_LENGTH_ARRAYS) && defined(VARIABLE_DECLARATION_AFTER_CODE)
 # define USE_SNTRUP761X25519 1
 #endif
-/* The ML-KEM768 imlementation similarly uses named struct initialisers */
-#ifdef NAMED_STRUCT_INITIALISERS
+/* The ML-KEM768 imlementation similarly uses C99 compound literals */
+#ifdef COMPOUND_LITERALS
 # define USE_MLKEM768X25519    1
 #endif
 #endif /* _DEFINES_H */