Good idea. I'll do that. Thanks for the review! Sue
On 11/30/11 10:41 AM, John Fischer wrote:
Sue, Looks good. You might update the function description that it now rejects service names that start with dash. John On 11/30/11 10:01 AM, Sue Sohn wrote:Can I get a code review of the changes for: 7114778 Creating a service whose name starts with dash ('-') leads to problems Webrev: https://cr.opensolaris.org/action/browse/caiman/sohn/7114778/webrev.7114778 Testing: Verified that service can no longer be created with name that starts with dash. Verified that service can still be created starting with other valid characters. Wrote new unit test for validate_service_name(). Thanks, Sue _______________________________________________ caiman-discuss mailing list [email protected] http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
_______________________________________________ caiman-discuss mailing list [email protected] http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
