8 May
2017
8 May
'17
9:35 a.m.
This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "An utility library for Open Source Mobile Communications".
The branch, master has been updated
via 85a6af213e8ccc0eb177b994db4137cf5b3089a4 (commit)
from a8a5819155323be6fab2c874d4448c0c9510331e (commit)
Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.
- Log -----------------------------------------------------------------
http://cgit.osmocom.org/libosmocore/commit/?id=85a6af213e8ccc0eb177b994db41…
commit 85a6af213e8ccc0eb177b994db4137cf5b3089a4
Author: Philipp Maier <pmaier(a)sysmocom.de>
Date: Fri Apr 28 10:55:05 2017 +0200
gsm0808: fix control flow issue
Coverity Scan reported a control flow issue in line 206:
CID 166898: Control flow issues (DEADCODE)
The second branch of the if statement can not be reached. The
purpose of the second if branch was to filter out zero length
elements if the header states that it is a non extended speech
codec type. This makes no sense, since the header needs at
least one byte.
This patch removes the second if branch, zero length elements
are catched by the already existing zero length check at the
beginning of the function
Change-Id: I89751fc0d598734c64ef1fdced75b7c4fa77c616
-----------------------------------------------------------------------
Summary of changes:
src/gsm/gsm0808_utils.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
hooks/post-receive
--
An utility library for Open Source Mobile Communications