Peter Memishian wrote: > > > > > * 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.
That's fine with me. -Seb
