diff options
| author | Pau Espin Pedrol <pespin@sysmocom.de> | 2018-03-26 18:09:19 +0200 |
|---|---|---|
| committer | Pau Espin Pedrol <pespin@sysmocom.de> | 2018-03-26 18:10:15 +0200 |
| commit | 39fb1fb916effc6fc603555238c0d08d4673ffa6 (patch) | |
| tree | 0a6638fcc44a8c0d61a25cf9a02601a58eaf06a8 | |
| parent | 052364d906748814ff1c2018fd77274020aa1e59 (diff) | |
libbsc: nokia_site: Fix uninitialized return val
Fixes following compilation warning:
bts_nokia_site.c:1248:9: error: ‘ret’ may be used uninitialized in this function [-Werror=maybe-uninitialized]
return ret;
This could happen if parameter len is passed with value 0.
If we reach the end of the function, it means everything went fine, so
we can return 0 instead of using ret, which is only used to store error
code before returning.
Change-Id: Ic60c8c632a6ddadb8f197d56cc68c2dbfc121d05
| -rw-r--r-- | src/libbsc/bts_nokia_site.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/libbsc/bts_nokia_site.c b/src/libbsc/bts_nokia_site.c index 6221e5f33..67281bbf1 100644 --- a/src/libbsc/bts_nokia_site.c +++ b/src/libbsc/bts_nokia_site.c @@ -1245,7 +1245,8 @@ static int abis_nm_send_multi_segments(struct gsm_bts *bts, uint8_t msg_type, data += len_to_send; seq++; } - return ret; + + return 0; } /* TODO: put in a separate file ? */ |