#### Pre-Submission Checklist
- [x] Commit message has the format required by CONTRIBUTING guide
- [x] Commits are split per component (core, individual modules, libs, utils, 
...)
- [x] Each component has a single commit (if not, squash them into one commit)
- [x] 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
- [x] 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:
- [x] PR should be backported to stable branches
- [x] Tested changes locally
- [ ] Related to issue 

#### Description

In `send_rtpp_command()` if socket operations fail with an error other than 
`EINTR` or `ENOBUFS`, the actual error is not reported - we just get a generic 
"Can't send command to/read reply from RTPEngine".

Other places in the module report `errno` and `strerror(errno)` using the log 
format "`(%s:%d)`" so added that to the error log.

Also fixed `bind_force_send_ip()` error log to use the same semantic as the 
other places.

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

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

-- Commit Summary --

  * rtpengine: show errno and error string in more places where it is useful

-- File Changes --

    M src/modules/rtpengine/rtpengine.c (14)

-- Patch Links --

https://github.com/kamailio/kamailio/pull/3899.patch
https://github.com/kamailio/kamailio/pull/3899.diff

-- 
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3899
You are receiving this because you are subscribed to this thread.

Message ID: <kamailio/kamailio/pull/3...@github.com>
_______________________________________________
Kamailio (SER) - Development Mailing List
To unsubscribe send an email to sr-dev-le...@lists.kamailio.org

Reply via email to