I wrote:
> Hmm, changing it like that seems likely to have a lot of unexpected
> side-effects.  What I had in mind was to change the 
>             if (te->nDeps != 1 ||
>                 TocIDRequired(AH, te->dependencies[0]) == 0)
>                 return 0;
> bit so that it would loop through the TE's dependencies to see
> if any one of them is a required non-ACL TE.  We have to relax
> the "te->nDeps != 1" restriction, but we still want to check
> that there is a parent object that is being restored.
I improved the patch along those lines and pushed it.
            regards, tom lane