[master] c3bd169 Silence a Coverity non-defect

[Dridi Boukelmoune]

On Thu, May 24, 2018 at 10:29 AM, Poul-Henning Kamp <phk at phk.freebsd.dk> wrote:
> --------
> In message <CABoVN9DFr5PwG+8BjmcB7oSgoXmudJ_4htoWyODJfOtJOYxZMg at mail.gmail.com>, Dridi Boukelmoune writes:
>
>>> --- a/bin/varnishd/cache/cache_panic.c
>>> +++ b/bin/varnishd/cache/cache_panic.c
>>> @@ -401,6 +401,7 @@ pan_busyobj(struct vsb *vsb, const struct busyobj *bo)
>>>         VSB_indent(vsb, 2);
>>>         PAN_CheckMagic(vsb, bo, BUSYOBJ_MAGIC);
>>>         pan_ws(vsb, bo->ws);
>>> +       AN(bo->vfc);
>>
>>Is it safe to put an assert here? Shouldn't we guard accesses to
>>bo->vfc if Nils thought it could be NULL?
>
> Actually, it's more that we don't want the panic code to panic
> because something is wrong:  We want it to report with
> precision that something is wrong.

That's exactly why I'm asking!