Could I please get a code review for a small change to fix bug 7807 ?

Bug:
-----
7807 installadm create-service crashes with no args
http://defect.opensolaris.org/bz/show_bug.cgi?id=7807


Webrev:
-------
http://cr.opensolaris.org/~joev/bug7807/


The modules affected and tested:
--------------------------------
installadm

Testing done:
--------------

1.
I built a stand alone C program to exercise the changed function, 
strip_ending_slashes. The stand along program passed various strings 
including NULL and the empty string to strip_ending_slashes.

2.
I built and ran installadm create-service on x86 with both no arguments 
and valid arguments.



Thank you!

Joe

Reply via email to