Reduce the scope of a variable, and remove pointless memsets (the same memory is immediately overwritten by the subsequent memcpy)
Fix an issue with memory not being cleaned up; according to libnice the string output needs to be cleaned up with g_free()
iceState and context_ are not initialized in the constructor.  Also move some stuff that's initialized in the constructor manually.
Release our context when we're done with it, otherwise it'll leak.
Fix constructor, initialize nice_, remove spurrious semicolons
nice_ doesn't need to be initialized anymore since it's a smart pointer.  remove unnecessary "friend" class.
Fix unsafe iterator behavior.  Once the iterator is erased, it is invalid and cannot be used.  It needs to be replaced by the iterator returned by erase().
Fix some super weird code in the dtls directory; this code is incorrectly freeing something it didn't allocate in the first place, although that would never happen because of an if-check that was wrong.  The memory it should be freeing shoul dbe freed with OPENSSL_free(), not regular free().
Remove unused variable, commented out code.
Fix memory leak in the case of an error.
Fix incorrect iterator usage, and update a log message to be correct.  Cannot erase the iterator currently being used; need to assign to return value of erase()
Fix potential memory leak.
Check for null before deleting.  Also zap some unused stuff.
Fix some memory leaks; stuff should begin its life as null always be deleted, and end its life as null.  Also zap unused pointer.
Fix memory leaks in the srtpSessionKey handling; it was allocating memory that was not being free'd.  Changed it so teh SrtpSessionKeys class handles all allocation and deallocation.
Remove a bunch of unnecessary memset that are immediately followed by a memcpy; the memcpy is going to overwrite anything in that memory location, so the memset is pointless.
Fix memory leaks, rename variables so it's more clear they're member variables of the class, remove unnecessary memcpy (already happens above--doing it a second time does nothing, unless there's some other thread overwriting that buffer.  If that's the case, a mutex is the appropriate fix, not copying the buffer repeatedly)
Fix a crash that could happen due to unpackagedBuffer_ ptr being incremented; when it was free'd and didn't point to the original location, it blows up. Also fix a few more leaks.
Fix some leaks, remove exit() calls, fix incorrect scoped_ptr usage.  Scoped_ptr calls delete under the hood, so using .reset() with malloc will cause both a leak and a memory allocation lib mis-match, resulting in undefined behavior.  Exit calls should never be used inside an application; it causes the entire app to stop running.
Rename a variable to make it more clear it's member, fix some leaks in the videomixer code.