Skip to content

Commit

Permalink
Partially revert "tree-wide: add missing magic checks"
Browse files Browse the repository at this point in the history
This reverts commit 328540c in the
lib/libvarnishapi/vsc.c file where no magic check could be added to
replace AN(vsm) occurrences.

We should probably look for more potential net loss of assertions from
that commit.
  • Loading branch information
dridi committed Jun 5, 2024
1 parent d3db146 commit e444efa
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions lib/libvarnishapi/vsc.c
Original file line number Diff line number Diff line change
Expand Up @@ -319,6 +319,7 @@ vsc_del_seg(const struct vsc *vsc, struct vsm *vsm, struct vsc_seg **spp)
struct vsc_seg *sp;

CHECK_OBJ_NOTNULL(vsc, VSC_MAGIC);
AN(vsm);
TAKE_OBJ_NOTNULL(sp, spp, VSC_SEG_MAGIC);
AZ(VSM_Unmap(vsm, sp->fantom));
if (sp->vj != NULL) {
Expand All @@ -342,6 +343,7 @@ vsc_add_seg(const struct vsc *vsc, struct vsm *vsm, const struct vsm_fantom *fp)
struct vsc_pt *pp;

CHECK_OBJ_NOTNULL(vsc, VSC_MAGIC);
AN(vsm);

ALLOC_OBJ(sp, VSC_SEG_MAGIC);
AN(sp);
Expand Down

0 comments on commit e444efa

Please sign in to comment.