[Spice-commits] server/utils.c
Frediano Ziglio
fziglio at kemper.freedesktop.org
Thu Nov 9 10:58:38 UTC 2017
server/utils.c | 10 ++++++++++
1 file changed, 10 insertions(+)
New commits:
commit fb3e061a8e05da5ab4acb4e3e676812cf7d4f1a0
Author: Frediano Ziglio <fziglio at redhat.com>
Date: Wed Nov 8 16:31:49 2017 +0000
utils: Check we list all channel names
This prevent future problems supporting new channels.
Signed-off-by: Frediano Ziglio <fziglio at redhat.com>
Acked-by: Victor Toso <victortoso at redhat.com>
diff --git a/server/utils.c b/server/utils.c
index ff1fc2d1..746132e5 100644
--- a/server/utils.c
+++ b/server/utils.c
@@ -20,6 +20,8 @@
#include <glib.h>
#include <spice/enums.h>
+#include <common/macros.h>
+
#include "utils.h"
int rgb32_data_has_alpha(int width, int height, size_t stride,
@@ -65,6 +67,14 @@ static const char *const channel_names[] = {
[ SPICE_CHANNEL_WEBDAV ] = "webdav",
};
+/* Make sure the last channel in the protocol has a name.
+ * We don't want to do this check in all cases as this would make code
+ * fail to compile if there are additional channels in the protocol so
+ * do this check only if ENABLE_EXTRA_CHECKS is enabled */
+#if ENABLE_EXTRA_CHECKS
+verify(G_N_ELEMENTS(channel_names) == SPICE_END_CHANNEL);
+#endif
+
/**
* red_channel_type_to_str:
*
More information about the Spice-commits
mailing list