OpenSolaris

Printable Version Enter a New Search
Bug ID 6737396
Synopsis Add an ASSERT in mac_client_remove_flow_from_list
State 10-Fix Delivered (Fix available in build)
Category:Subcategory kernel:gld
Keywords crossbow
Responsible Engineer Girish Moodalbail
Reported Against
Duplicate Of
Introduced In solaris_nevada
Commit to Fix snv_107
Fixed In snv_107
Release Fixed solaris_nevada(snv_107)
Related Bugs
Submit Date 14-August-2008
Last Update Date 28-January-2009
Description 
As part of the code review walk with Nicolas, in mac_client_remove_flow_from_list() of mac.c, add an ASSERT. 

static void
mac_client_remove_flow_from_list(mac_client_impl_t *mcip, flow_entry_t *flent)
{
	.......
	/* Should this be an ASSERT */ ====> Yes add an ASSERT!
	if (fe != NULL) {
	  	if (prev_fe == NULL) {
			/* Deleting the first node */
		  	mcip->mci_flent_list = fe->fe_client_next;
	.........
}
Work Around 
N/A
Comments 
N/A