> > > > * 131,132: Not that this uid and gid will ever change, but I'm > > thinking > > > > that it would be cleaner to obtain the uid and gid buy doing a > > > > getpwnam() lookup on the string "dladm" instead of hardcoding these > > > > values in the header file. > > > > > > > > > > My only concern is whether getpwnam() can work well during early phase > > > of system boot. I will test it out and see whether I can make this > > > change. > > > > I'd be nervous about that too. I recall there were some recently-fixed > > issues here -- e.g. 6630372. > > From my read of the bug report, there should not be any problem after > this bug is fixed. Is this right?
Yes, in that situation. But I'm not sure I'd want a critical system service relying on the existence or accuracy of /etc/passwd or /etc/group unless it's necessary -- and here, it doesn't seem necessary. -- meem
