d/freebsd-skq
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Since igb_detach() cleans up all the data structures that will be

Browse Source 
free'd by the functions following its call, we can simply return instead
of crashing and burning in the event of igb_detach() failing.

PR:		197139
Submitted by:	rupavath@juniper.net
MFC after:	2 weeks
This commit is contained in:
sbruno 2016-05-06 17:00:45 +00:00
parent 9d9aae4d84
commit e0f2b3bfc7
1 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
sys/dev/e1000/if_igb.c
View File

@ -659,7 +659,8 @@ igb_attach(device_t dev)
return (0); return (0);
err_late: err_late:
igb_detach(dev); if (igb_detach(dev) == 0) /* igb_detach() already did the cleanup */
return(error);
igb_free_transmit_structures(adapter); igb_free_transmit_structures(adapter);
igb_free_receive_structures(adapter); igb_free_receive_structures(adapter);
igb_release_hw_control(adapter); igb_release_hw_control(adapter);