More explicit and defensive code so if the init fails we know we handling this pointer safely, because  rc_config_init will free the pointer if there was a problem, but it will not set the pointer to NULL.

Appears to have been a copy paste error because this doesn't do what the api claims, and it also leaks the reference to the new node.

FS-5453 --resolve this is a different issue, we just have to hangup the loopback channel so the loopback and the sip don't both execute the extension

FS-5402 --resolve the problem is ent originate has many channels and many causes to parse so we need to check it against all the causes

Theoretical memory leak where if attr_init failed, then check would have still been malloc'd but not used.

Minor code cleanup. calling va_start without calling va_end doesn't leak does it? Either way, it's fixed now.

full_contact must be freed by sofia. There isn't a chance of this having blown anything up, but Coverity is right that this should be initialized with NULL to be safe against future refactors.

My bad. some how a make didn't catch this issue after a cleanup refactor. In this instance buf is being treated as an 8 byte number, not a pointer.

Coverity reported an Uninitialized pointer read. Upon further digging it appears that there is a code path where incoming packets can come in out of order, so this section of code tries to see if it can find the missed packets. As per FS-5202 there is a case where under heavy load the packet exists, and has most of the packet parsed, but still has a NULL pointer for the packet buffer. These two lines would at least help detect the edge case.

Edge case for when you are subscribing to presence on a valet parking lot. In which case there would be a small leak.

Possible leak if someone were to try and add extra headers to a channel that had either an inbound info on that session or the session were to be deflected. I don't see a way that these extra headers could be added outside of a C module subscribing to the session states, but just to make sure I'm adding these so that just like other extra header locations it's cleaned up.