Follow @Openwall on Twitter for new release announcements and other news
[<prev] [next>] [day] [month] [year] [list]
Date: Sat, 28 Dec 2019 05:38:14 -0600
From: "A. Wilcox" <AWilcox@...cox-Tech.com>
To: arnd@...db.de,
	musl@...ts.openwall.com,
	baolin.wang@...aro.org
Cc: "A. Wilcox" <AWilcox@...cox-Tech.com>
Subject: [PATCH] ALSA: Fix definition of __time_pad

The uapi definition of __time_pad was missing a closing brace, causing
compile errors in ex. alsa-lib:

In file included from ../../include/local.h:146,
                 from control_local.h:22,
                 from tlv.c:36:
/usr/include/sound/asound.h:471:66: error: expected ‘:’, ‘,’, ‘;’, ‘}’ or ‘__attribute__’ before ‘__time_pad’
 typedef struct { unsigned char pad[sizeof(time_t) - sizeof(int)] __time_pad;
                                                                  ^~~~~~~~~~

Fixes: df4d6b0e01b8 ("ALSA: Avoid using timespec for struct snd_pcm_status")
Signed-off-by: A. Wilcox <AWilcox@...cox-Tech.com>
---
 include/uapi/sound/asound.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/include/uapi/sound/asound.h b/include/uapi/sound/asound.h
index e6a958b8aff1..b0e5c45204d4 100644
--- a/include/uapi/sound/asound.h
+++ b/include/uapi/sound/asound.h
@@ -471,7 +471,7 @@ enum {
 
 #ifndef __KERNEL__
 /* explicit padding avoids incompatibility between i386 and x86-64 */
-typedef struct { unsigned char pad[sizeof(time_t) - sizeof(int)] __time_pad;
+typedef struct { unsigned char pad[sizeof(time_t) - sizeof(int)] } __time_pad;
 
 struct snd_pcm_status {
 	snd_pcm_state_t state;		/* stream state */
-- 
2.24.1

Powered by blists - more mailing lists

Confused about mailing lists and their use? Read about mailing lists on Wikipedia and check out these guidelines on proper formatting of your messages.