Changeset 807 for pjproject/trunk/pjmedia/src/pjmedia-codec/speex_codec.c

Timestamp:

Nov 19, 2006 8:28:34 PM (17 years ago)

Author:

bennylp

Message:

Fixed memory leaks bug in speex codec

File:

• pjproject/trunk/pjmedia/src/pjmedia-codec/speex_codec.c (modified) (2 diffs)

pjproject/trunk/pjmedia/src/pjmedia-codec/speex_codec.c

@@ -311 +311 @@
     }
 
+    pj_mutex_lock(spx_factory.mutex);
+
     /* We don't want to deinit if there's outstanding codec. */
-    pj_mutex_lock(spx_factory.mutex);
+    /* This is silly, as we'll always have codec in the list if
+       we ever allocate a codec! A better behavior maybe is to 
+       deallocate all codecs in the list.
     if (!pj_list_empty(&spx_factory.codec_list)) {
         pj_mutex_unlock(spx_factory.mutex);
         return PJ_EBUSY;
     }
+    */
 
     /* Get the codec manager. */
@@ -743 +748 @@
     }
 
-    /* Initialization of the structure that holds the bits */
-    speex_bits_init(&spx->dec_bits);
-
     /* Copy the data into the bit-stream struct */
     speex_bits_read_from(&spx->dec_bits, input->buf, input->size);