In one place this is already done but this makes sure we show the bad
attribute in all cases where a non conforming attribute is found.
OK?
--
:wq Claudio
Index: rrdp_delta.c
===================================================================
RCS file: /cvs/src/usr.sbin/rpki-client/rrdp_delta.c,v
retrieving revision 1.4
diff -u -p -r1.4 rrdp_delta.c
--- rrdp_delta.c 28 Oct 2021 11:57:00 -0000 1.4
+++ rrdp_delta.c 3 Nov 2021 11:52:42 -0000
@@ -87,7 +87,7 @@ start_delta_elem(struct delta_xml *dxml,
continue;
}
PARSE_FAIL(p, "parse failed - non conforming "
- "attribute found in delta elem");
+ "attribute '%s' found in delta elem", attr[i]);
}
if (!(has_xmlns && dxml->version && dxml->session_id && dxml->serial))
PARSE_FAIL(p, "parse failed - incomplete delta attributes");
@@ -136,7 +136,7 @@ start_publish_withdraw_elem(struct delta
continue;
}
PARSE_FAIL(p, "parse failed - non conforming "
- "attribute found in publish/withdraw elem");
+ "attribute '%s' found in publish/withdraw elem", attr[i]);
}
if (hasUri != 1)
PARSE_FAIL(p,
Index: rrdp_snapshot.c
===================================================================
RCS file: /cvs/src/usr.sbin/rpki-client/rrdp_snapshot.c,v
retrieving revision 1.3
diff -u -p -r1.3 rrdp_snapshot.c
--- rrdp_snapshot.c 28 Oct 2021 11:57:00 -0000 1.3
+++ rrdp_snapshot.c 3 Nov 2021 11:51:45 -0000
@@ -80,7 +80,7 @@ start_snapshot_elem(struct snapshot_xml
}
PARSE_FAIL(p,
"parse failed - non conforming "
- "attribute found in snapshot elem");
+ "attribute '%s' found in snapshot elem", attr[i]);
}
if (!(has_xmlns && sxml->version && sxml->session_id && sxml->serial))
PARSE_FAIL(p,