Mercurial > pidgin
diff src/protocols/msn/session.c @ 13693:510b7b7dd645
[gaim-migrate @ 16096]
Part of SF Patch #1437836 from Sadrul
'3. replace a bunch of "if (a) free(a);" with "free(a);"'
My comment in the patch:
"I'm accepting this, even though it doesn't really do much. In the future, however, I would suggest avoiding creating these type of changes unless you're already editing the code directly around them."
committer: Tailor Script <tailor@pidgin.im>
| author | Richard Laager <rlaager@wiktel.com> |
|---|---|
| date | Tue, 25 Apr 2006 17:38:43 +0000 |
| parents | 8872789902a1 |
| children | 4e44ecb866bd 995aea35b05c |
line wrap: on
line diff
--- a/src/protocols/msn/session.c Tue Apr 25 17:27:17 2006 +0000 +++ b/src/protocols/msn/session.c Tue Apr 25 17:38:43 2006 +0000 @@ -70,17 +70,10 @@ msn_userlist_destroy(session->userlist); - if (session->passport_info.kv != NULL) - g_free(session->passport_info.kv); - - if (session->passport_info.sid != NULL) - g_free(session->passport_info.sid); - - if (session->passport_info.mspauth != NULL) - g_free(session->passport_info.mspauth); - - if (session->passport_info.client_ip != NULL) - g_free(session->passport_info.client_ip); + g_free(session->passport_info.kv); + g_free(session->passport_info.sid); + g_free(session->passport_info.mspauth); + g_free(session->passport_info.client_ip); if (session->passport_info.file != NULL) {