diff options
| author | João Valverde <joao.valverde@tecnico.ulisboa.pt> | 2015-12-14 06:42:41 +0000 |
|---|---|---|
| committer | João Valverde <j@v6e.pt> | 2016-02-19 15:48:16 +0000 |
| commit | 5fec8fa74619a69e3faeca01fd483157d39f7b13 (patch) | |
| tree | a216be6fe5393263a0145553cb75d96b053b8c4a /epan/dissectors/packet-per.c | |
| parent | 0200cff989e129c658438a0104c2bb3e9fe55e27 (diff) | |
BER/PER: Add dissector assertions to squelch scan-build bugs
packet-per.c:959:6: warning: Access to field 'aligned' results in a dereference of a null pointer (loaded from variable 'actx')
packet-ber.c:1606:29: warning: Access to field 'pinfo' results in a dereference of a null pointer (loaded from variable 'actx')
packet-ber.c:1612:24: warning: Access to field 'created_item' results in a dereference of a null pointer (loaded from variable 'actx')
packet-ber.c:3156:41: warning: Access to field 'pinfo' results in a dereference of a null pointer (loaded from variable 'actx')
packet-ber.c:3182:24: warning: Access to field 'created_item' results in a dereference of a null pointer (loaded from variable 'actx')
Change-Id: Ibae00dc29a869701fe903a5b0c9944279aaa3df7
Reviewed-on: https://code.wireshark.org/review/13936
Petri-Dish: João Valverde <j@v6e.pt>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
Reviewed-by: João Valverde <j@v6e.pt>
Diffstat (limited to 'epan/dissectors/packet-per.c')
| -rw-r--r-- | epan/dissectors/packet-per.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/epan/dissectors/packet-per.c b/epan/dissectors/packet-per.c index eed095eb6d..ca02072bbc 100644 --- a/epan/dissectors/packet-per.c +++ b/epan/dissectors/packet-per.c @@ -2399,6 +2399,8 @@ gboolean get_size_constraint_from_stack(asn1_ctx_t *actx, const gchar *name, int static int dissect_per_T_direct_reference(tvbuff_t *tvb, int offset, asn1_ctx_t *actx, proto_tree *tree, int hf_index) { + + DISSECTOR_ASSERT(actx); offset = dissect_per_object_identifier_str(tvb, offset, actx, tree, hf_index, &actx->external.direct_reference); actx->external.direct_ref_present = TRUE; |