[Home]

Summary:ASTERISK-27990: res_rtp_asterisk: Requires OpenSSL in Developer Mode.
Reporter:Alexander Traud (traud)Labels:patch
Date Opened:2018-07-27 07:45:36Date Closed:2018-08-01 04:24:30
Priority:MajorRegression?
Status:Closed/CompleteComponents:Resources/res_rtp_asterisk
Versions:13.22.0 15.5.0 Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) openssl_no_srtp_fix.patch
Description:This is a regression, introduced by myself in [Gerrit 9184|https://gerrit.asterisk.org/9184]. When Asterisk was configured with {{--enable-dev-mode}}, and the headers of OpenSSL are not installed, {{make}} fails with{code}res_rtp_asterisk.c:273:35: error: "OPENSSL_VERSION_NUMBER" is not defined, evaluates to 0 [-Werror=undef]{code}This happens, because {{HAVE_OPENSSL}} is not checked. The attached patch fixes this issue.
Comments:By: Asterisk Team (asteriskteam) 2018-07-27 07:45:38.001-0500

Thanks for creating a report! The issue has entered the triage process. That means the issue will wait in this status until a Bug Marshal has an opportunity to review the issue. Once the issue has been reviewed you will receive comments regarding the next steps towards resolution.

A good first step is for you to review the [Asterisk Issue Guidelines|https://wiki.asterisk.org/wiki/display/AST/Asterisk+Issue+Guidelines] if you haven't already. The guidelines detail what is expected from an Asterisk issue report.

Then, if you are submitting a patch, please review the [Patch Contribution Process|https://wiki.asterisk.org/wiki/display/AST/Patch+Contribution+Process].

By: Friendly Automation (friendly-automation) 2018-08-01 04:24:31.367-0500

Change 9736 merged by Joshua Colp:
res_rtp_asterisk: In Developer Mode, do not require OpenSSL.

[https://gerrit.asterisk.org/9736|https://gerrit.asterisk.org/9736]

By: Friendly Automation (friendly-automation) 2018-08-01 04:24:39.924-0500

Change 9737 merged by Joshua Colp:
res_rtp_asterisk: In Developer Mode, do not require OpenSSL.

[https://gerrit.asterisk.org/9737|https://gerrit.asterisk.org/9737]

By: Friendly Automation (friendly-automation) 2018-08-01 04:24:48.737-0500

Change 9739 merged by Joshua Colp:
res_rtp_asterisk: In Developer Mode, do not require OpenSSL.

[https://gerrit.asterisk.org/9739|https://gerrit.asterisk.org/9739]

By: Friendly Automation (friendly-automation) 2018-08-01 04:24:56.451-0500

Change 9738 merged by Joshua Colp:
res_rtp_asterisk: In Developer Mode, do not require OpenSSL.

[https://gerrit.asterisk.org/9738|https://gerrit.asterisk.org/9738]