diff options
| author | Neels Janosch Hofmeyr <nhofmeyr@sysmocom.de> | 2023-01-02 14:05:36 +0100 |
|---|---|---|
| committer | Neels Janosch Hofmeyr <nhofmeyr@sysmocom.de> | 2023-01-02 14:05:45 +0100 |
| commit | 9c96c3986f03ab3b776cbf8ab7db624ec704c2a3 (patch) | |
| tree | 79346442fb390f415f7d84756a93ce1a654bdb1f | |
| parent | bf8e49a1bde4862eda5c32192ffb696b9963d90a (diff) | |
silence compiler warning in gtlv_dec_enc.c
This was reported with gcc version 12.2.0:
/git/libosmo-pfcp/src/libosmo-gtlv/gtlv_dec_enc.c: In function
'osmo_gtlvs_decode_unordered':
/git/libosmo-pfcp/src/libosmo-gtlv/gtlv_dec_enc.c:237:42: warning:
'presence_flag_p' may be used uninitialized [-Wmaybe-uninitialized]
237 | *presence_flag_p = true;
| ^
/git/libosmo-pfcp/src/libosmo-gtlv/gtlv_dec_enc.c:113:23: note:
'presence_flag_p' was declared here
113 | bool *presence_flag_p;
| ^~~~~~~~~~~~~~~
There is no actual code path that will use presence_flag_p
uninitialized, but it doesn't hurt to init with NULL.
Change-Id: I8f4c420f2182c607abb1ee5d1c8175eaeda904af
| -rw-r--r-- | src/libosmo-gtlv/gtlv_dec_enc.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/libosmo-gtlv/gtlv_dec_enc.c b/src/libosmo-gtlv/gtlv_dec_enc.c index 37e8a51..bbd9db7 100644 --- a/src/libosmo-gtlv/gtlv_dec_enc.c +++ b/src/libosmo-gtlv/gtlv_dec_enc.c @@ -110,7 +110,7 @@ static int osmo_gtlvs_decode_unordered(void *decoded_struct, size_t decoded_stru * any). */ for (;;) { int rc; - bool *presence_flag_p; + bool *presence_flag_p = NULL; unsigned int memb_next_array_idx; unsigned int memb_ofs; unsigned int ie_max_allowed_count; |