Attachment #205221 for bug #238707

FreeBSD Bugzilla – Attachment #205221: Patch to move the unlock of the rtentry just before the call of rt_notifydelete for bug #238707

Home
| New
| Browse
| Search
|

[?]
| Reports
| New Account
| Log In
Remember [x]
| Forgot Password
Login: [x]

View | Details | Raw Unified | Return to bug 238707
Collapse All | Expand All




		if (error != 0)
			return (error);

		RT_UNLOCK(rt);
		rt_notifydelete(rt, info);

		/*

		 * but it's up to it to free the rtentry as we won't be
		 * doing it.
		 */
		if (ret_nrt)
			*ret_nrt = rt;
		else
			RTFREE(rt);
			RTFREE_LOCKED(rt);
		break;
	case RTM_RESOLVE:
		/*

Return to bug 238707

Lines 1600-1605 rtrequest1_fib(int req, struct rt_addrinfo info, struct rtentry *ret_nrt, Link Here

(-)sys/net/route.c (-4 / +4 lines)
1600	if (error != 0)	1600	if (error != 0)
1601	return (error);	1601	return (error);
1602		1602
		1603	RT_UNLOCK(rt);
1603	rt_notifydelete(rt, info);	1604	rt_notifydelete(rt, info);
1604		1605
1605	/*	1606	/*
Lines 1607-1617 rtrequest1_fib(int req, struct rt_addrinfo info, struct rtentry *ret_nrt, Link Here
1607	* but it's up to it to free the rtentry as we won't be	1608	* but it's up to it to free the rtentry as we won't be
1608	* doing it.	1609	* doing it.
1609	*/	1610	*/
1610	if (ret_nrt) {	1611	if (ret_nrt)
1611	*ret_nrt = rt;	1612	*ret_nrt = rt;
1612	RT_UNLOCK(rt);	1613	else
1613	} else	1614	RTFREE(rt);
1614	RTFREE_LOCKED(rt);
1615	break;	1615	break;
1616	case RTM_RESOLVE:	1616	case RTM_RESOLVE:
1617	/*	1617	/*