> - cleanup: > +cleanup: > kfree(reply); > + if (msg) > +out: > + i2o_msg_nop(c, msg); Put the label before the if. Much saner that way > kfree(reply); > + if (msg) > +out: > + i2o_msg_nop(c, msg); > return rcode; Ditto