Tony, Thanks for you review. All your comments are pertinent. I'll be fixing them today.
Antonello Tony Nguyen wrote: > Sorry for the late response but reviewing scf_tmpl.c took longer than I > expected. Just a few minor comments. > > -tony > > scf_tmpl.c: > > 1068 - Isn't this a redundant assertion? > > 1956 - Shouldn't this assertion be done prior to obtaining the handle? > > 4198 - Need to free(encoded)? > > 5067, 5071 and others - We don't need to set these to NULL, do we? > _______________________________________________ > smf-discuss mailing list > smf-discuss at opensolaris.org
