summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorFrediano Ziglio <fziglio@redhat.com>2016-01-28 14:40:06 +0000
committerFrediano Ziglio <fziglio@redhat.com>2016-03-11 10:16:22 +0000
commit53ee80bd7d1d98d87654c57098722532df55dd1b (patch)
treed358ce6d7fe4205ce9f3973202f7aca33558082d
parent8af947e7b2062dda564bd9f6aacf5bc6b3da5e51 (diff)
downloadspice-common-master.tar.gz
spice-common-master.tar.xz
spice-common-master.zip
Cap logging level to the valid boundsHEADmaster
Avoid overflows using its values. The patch was originally written by Christophe Fergeau Acked-by: Victor Toso <victortoso@redhat.com>
-rw-r--r--common/log.c3
1 files changed, 2 insertions, 1 deletions
diff --git a/common/log.c b/common/log.c
index a9bbebc..8d47cb6 100644
--- a/common/log.c
+++ b/common/log.c
@@ -50,7 +50,8 @@ static GLogLevelFlags spice_log_level_to_glib(SpiceLogLevel level)
[ SPICE_LOG_LEVEL_INFO ] = G_LOG_LEVEL_INFO,
[ SPICE_LOG_LEVEL_DEBUG ] = G_LOG_LEVEL_DEBUG,
};
- g_return_val_if_fail ((level >= 0) || (level < G_N_ELEMENTS(glib_levels)), 0);
+ g_return_val_if_fail (level >= 0, G_LOG_LEVEL_ERROR);
+ g_return_val_if_fail (level < G_N_ELEMENTS(glib_levels), G_LOG_LEVEL_DEBUG);
return glib_levels[level];
}