From: David Feurle <david.feurle@sodgeit.de>
Date: Sat, 7 Feb 2015 09:49:07 +0000 (+0100)
Subject: stun: Use dynamic array instead of stack allocated array
X-Git-Tag: 0.1.11~20
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=4723a27091166eee40b8ac5d4004df373b5736c2;p=platform%2Fupstream%2Flibnice.git

stun: Use dynamic array instead of stack allocated array

Dynamic on-stack arrays are not supported in Visual Studio.

This has the downside of introducing an extra memory allocation into
libstun, but it’s on a debug path so should be harmless.
---

diff --git a/stun/debug.c b/stun/debug.c
index f3a55bb..3d579db 100644
--- a/stun/debug.c
+++ b/stun/debug.c
@@ -79,11 +79,12 @@ void stun_debug_bytes (const char *prefix, const void *data, size_t len)
 {
   size_t i;
   size_t prefix_len = strlen (prefix);
-  char bytes[prefix_len + 2 + (len * 2) + 1];
+  char *bytes;
 
   if (!debug_enabled)
     return;
 
+  bytes = malloc (prefix_len + 2 + (len * 2) + 1);
   bytes[0] = 0;
   strcpy (bytes, prefix);
   strcpy (bytes + prefix_len, "0x");
@@ -92,6 +93,7 @@ void stun_debug_bytes (const char *prefix, const void *data, size_t len)
     sprintf (bytes + prefix_len + 2 + (i * 2), "%02x", ((const unsigned char *)data)[i]);
 
   stun_debug ("%s", bytes);
+  free (bytes);
 }