Fix some issues scanbuild found in the xmpp protocol plugin

Review Request #711 — Created June 2, 2021 and submitted

Information

pidgin/pidgin
release-2.x.y
0b139e724ace

Reviewers

Fix some issues scanbuild found in the xmpp protocol plugin
Verified that scanbuild no longer detected the errors.
Description From Last Updated

There's no diff?

QuLogicQuLogic

This leaks as well.

QuLogicQuLogic
grim
grim
QuLogic
  1. 
      
  2. Show all issues

    There's no diff?

    1. bleh i accidentially discarded it in a draft state so it probably ditched it and i didn't check.. updating.

  3. 
      
grim
QuLogic
  1. 
      
  2. libpurple/protocols/jabber/auth_scram.c (Diff revision 1)
     
     
    Show all issues

    This leaks as well.

  3. 
      
grim
QuLogic
  1. Ship It!
  2. 
      
grim
Review request changed
Status:
Completed
jrchamp
  1. 
      
  2. grim, the reason this one wasn't triggering a memory leak according to scanbuild is problem because session->state is ALWAYS set to UNINIT by the calling function that allocated the memory.

    1. Sorry, that should have been "probably", not "problem".
  3.