[spice-gtk v1 8/9] debug: don't segfault on CHANNEL_DEBUG

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On a test environment, not always we have the channel present on
CHANNEL_DEBUG which could cause a crash in the code. Just defaults to
"no-channel" should be enough.
---
 src/spice-channel-priv.h | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/src/spice-channel-priv.h b/src/spice-channel-priv.h
index 526b661..f0bb017 100644
--- a/src/spice-channel-priv.h
+++ b/src/spice-channel-priv.h
@@ -41,7 +41,9 @@ G_BEGIN_DECLS
 #define MAX_SPICE_DATA_HEADER_SIZE sizeof(SpiceDataHeader)
 
 #define CHANNEL_DEBUG(channel, fmt, ...) \
-    SPICE_DEBUG("%s: " fmt, SPICE_CHANNEL(channel)->priv->name, ## __VA_ARGS__)
+    SPICE_DEBUG("%s: " fmt, \
+            (channel) ? SPICE_CHANNEL(channel)->priv->name : "no-channel", \
+            ## __VA_ARGS__)
 
 struct _SpiceMsgOut {
     int                   refcount;
-- 
2.5.5

_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]