illumos gate

See Comments.

xxxxx@xxxxx.com 2004-12-21 07:26:51 GMT
[dp: this is marked bite size, so relocating comments into description]

bustos 2004-12-20

print_dependency_reasons() in svcs/explain.c has

1576         for (spp = uu_list_first(svcp->causes);
   1577             spp != NULL && spp->svcp != svcp;
   1578             spp = uu_list_next(svcp->causes, spp)) {
   1579                 determine_summary(spp->svcp);

I think the while clause might be wrong, and maybe should be

for (spp = uu_list_first(svcp->causes);
        spp != NULL;
        spp = uu_list_next(svcp->causes, spp)) {
        if (spp->svcp == svcp)
            continue;

determine_summary(spp->svcp);

If so, then svcs -x might miss reasons after the self-pointer. The question
is whether any other reasons can be placed after a self-pointer. This
needs to be investigated.

xxxxx@xxxxx.com 2004-12-21 07:26:51 GMT