<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Thu, May 24, 2018 at 10:22 AM, Dridi Boukelmoune <span dir="ltr"><<a href="mailto:dridi@varni.sh" target="_blank">dridi@varni.sh</a>></span> wrote:<span class=""></span><br><span class=""></span><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">
> --- a/bin/varnishd/cache/cache_<wbr>panic.c<br>
> +++ b/bin/varnishd/cache/cache_<wbr>panic.c<br>
> @@ -401,6 +401,7 @@ pan_busyobj(struct vsb *vsb, const struct busyobj *bo)<br>
>         VSB_indent(vsb, 2);<br>
>         PAN_CheckMagic(vsb, bo, BUSYOBJ_MAGIC);<br>
>         pan_ws(vsb, bo->ws);<br>
> +       AN(bo->vfc);<br>
<br>
</span>Is it safe to put an assert here? Shouldn't we guard accesses to<br>
bo->vfc if Nils thought it could be NULL?<br clear="all"><br></blockquote><div><br></div><div>bo->vfc has the same lifetime as the bo itself, so that should be OK. <br></div></div><br>-- <br><div class="gmail_signature" data-smartmail="gmail_signature">Dag Haavi Finstad<br>Software Developer | Varnish Software<br>Mobile: +47 476 64 134<br>We Make Websites Fly!</div>
</div></div>