Darren Reed writes: > >Please make sure that you're listed as the RE on all of the bugs, > >you've got a filled out 'Evaluation' field, and the state is > >'fix-in-progress.' > > > > > > Yup, I'll do that...I suppose it's ok for me to "accept" 6678354 myself?
Sure. (Why not ... ?) > >revarp.c > > > > 374: this return leaks the 'dh' handle. (Since this rarely happens, > > couldn't this just be another test near line 386?) > > > > > > I had a think about this... > The concern was whether the early return saved on a possible > error from dlpi_info(). Given the nature of what I observed, > "ether ff", it's clear that this isn't a concern, so I'm happy to > merge the check into line 386. OK. > >ifconfig.c > > > > 2884: while you're here doing cosmetic stuff, lifr_metric isn't > > right. This should be lifr_mtu. (Cut-n-paste, I guess.) > > > > > > Yup. Thanks for cleaning up some cruft. -- James Carlson, Solaris Networking <[EMAIL PROTECTED]> Sun Microsystems / 35 Network Drive 71.232W Vox +1 781 442 2084 MS UBUR02-212 / Burlington MA 01803-2757 42.496N Fax +1 781 442 1677 _______________________________________________ networking-discuss mailing list [email protected]
