Pre-Submission Checklist

Commit message has the format required by CONTRIBUTING guide
Commits are split per component (core, individual modules, libs, utils, ...)
Each component has a single commit (if not, squash them into one commit)
No commits to README files for modules (changes must be done to docbook files
in doc/ subfolder, the README file is autogenerated)

Type Of Change

Small bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds new functionality)
Breaking change (fix or feature that would change existing functionality)

Checklist:

PR should be backported to stable branches
Tested changes locally
Related to issue #XXXX (replace XXXX with an open issue number)

Description

Fix the following issues found by Coverity:
3c86d68 fixes defect 85605
7f603ec fixes defect 86015
2772f09 fixes defects 85807 and 85651
d8fdcb3 fixes defects 153773, 85749 and 85702
3a9365f fixes defects 85541 and 85543
dbafb5a fixes defect 85484
535eb88 fixes defect 155203

You can view, comment on, or merge this pull request online at:

https://github.com/kamailio/kamailio/pull/1199

Commit Summary

carrierroute: avoid out-of-bounds access when resetting opts
carrierroute: check pointer before dereferencing it
carrieroute: remove logically dead code
carrierroute: avoid comparing size_t types with negative values
carrierroute: check return value for fseek
carrierroute: avoid truncating value returned by fgetc
carrierroute: avoid ovverruning a fixed sized buffer

File Changes

M src/modules/carrierroute/cr_config.c (125)
M src/modules/carrierroute/cr_db.c (6)
M src/modules/carrierroute/cr_fifo.c (11)
M src/modules/carrierroute/cr_func.c (14)
M src/modules/carrierroute/parser_carrierroute.c (13)

Patch Links:

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.