Hello,
I've got two issues with your patch:
1) You are introducing unnecessary function call for all users, this new
code-path should be evaluated only for Solaris users.
2) You are assuming that nginx is running as root, which isn't always true.

Attached is my patch for this issue, it would be nice if you could test it.

Best regards,
Piotr Sikora < piotr.sikora@frickle.com >
_______________________________________________
nginx-devel mailing list
nginx-devel@nginx.org
http://nginx.org/mailman/listinfo/nginx-devel