Karen Tung wrote: > Hi, > > Please review my changes for bug 2263: microroot overflows with latest > changes to IPS > > http://defect.opensolaris.org/bz/show_bug.cgi?id=2263 > > webrev is at: > > http://cr.opensolaris.org/~ktung/2263_fix/ > > Thanks, > > --Karen > > _______________________________________________ > caiman-discuss mailing list > caiman-discuss at opensolaris.org > http://mail.opensolaris.org/mailman/listinfo/caiman-discuss > Karen:
57: It is better to search for the whole string 'flush-content-cache-on-success' since it may not work if a new name starts with 'flush-content-cache' is added to the cfg_cache file in the future. 58: not aligned properly 63: The status value may be 0 since it was set in line 36 - Sundar