[master] c3bd169 Silence a Coverity non-defect
Poul-Henning Kamp
phk at phk.freebsd.dk
Thu May 24 08:29:37 UTC 2018
--------
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.
--
Poul-Henning Kamp | UNIX since Zilog Zeus 3.20
phk at FreeBSD.ORG | TCP/IP since RFC 956
FreeBSD committer | BSD since 4.3-tahoe
Never attribute to malice what can adequately be explained by incompetence.
More information about the varnish-commit
mailing list