Since igb_detach() cleans up all the data structures that will be
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