Aren
/
NetBSD
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

avoid reusing free'd pointer (Coverity 2613)

This commit is contained in:
manu 2006-10-02 21:51:33 +00:00
parent 1966cc3311
commit 9547d0f260
2 changed files with 7 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
crypto/dist/ipsec-tools/ChangeLog vendored
View File

@ -13,6 +13,7 @@
* src/racoon/algorithm.c: fix array overrun (Coverity)
* src/racoon/isakmp_ident.c: Remove dead code (Coverity)
* src/racoon/isakmp_inf.c: Check for NULL pointer (Coverity)
* src/racoon/isakmp_base.c: avoid reusing free'd pointer (Coverity)
2006-10-01 Emmanuel Dreyfus <manu@netbsd.org>
* src/racoon/isakmp.c: Avoid using NULL pointer (Coverity)

9
crypto/dist/ipsec-tools/src/racoon/isakmp_base.c vendored
View File

@ -1,4 +1,4 @@
/* $NetBSD: isakmp_base.c,v 1.6 2006/09/09 16:22:09 manu Exp $ */
/* $NetBSD: isakmp_base.c,v 1.7 2006/10/02 21:51:33 manu Exp $ */
/* $KAME: isakmp_base.c,v 1.49 2003/11/13 02:30:20 sakane Exp $ */
@ -1074,8 +1074,10 @@ base_r1send(iph1, msg)
/* send the packet, add to the schedule to resend */
iph1->retry_counter = iph1->rmconf->retry_counter;
if (isakmp_ph1resend(iph1) == -1)
if (isakmp_ph1resend(iph1) == -1) {
iph1 = NULL;
goto end;
}
/* the sending message is added to the received-list. */
if (add_recvdpkt(iph1->remote, iph1->local, iph1->sendbuf, msg) == -1) {
@ -1108,7 +1110,8 @@ end:
vfree(vid_dpd);
#endif
VPTRINIT(iph1->sa_ret);
if (iph1 != NULL)
VPTRINIT(iph1->sa_ret);
return error;
}