rtsp: Corrected the HTTP digest authorization computation.
authorPeter Kjellerstedt <pkj@axis.com>
Tue, 18 Aug 2009 14:49:58 +0000 (16:49 +0200)
committerPeter Kjellerstedt <pkj@axis.com>
Tue, 18 Aug 2009 14:50:58 +0000 (16:50 +0200)
Do not use sizeof() on an array passed as an argument to a function and
expect to get anything but the size of a pointer. As a result only the
first 4 (or 8) bytes of the response buffer were initialized to 0 in
auth_digest_compute_response() which caused it to return a string which
was not NUL-terminated...

gst-libs/gst/rtsp/gstrtspconnection.c

index 954ee98..58e5d2e 100644 (file)
@@ -933,7 +933,7 @@ auth_digest_compute_response (const gchar * method,
 
   g_checksum_update (md5_context, (const guchar *) hex_a2, 32);
   digest_string = g_checksum_get_string (md5_context);
-  memset (response, 0, sizeof (response));
+  memset (response, 0, 33);
   memcpy (response, digest_string, strlen (digest_string));
 
   g_checksum_free (md5_context);