[Asterisk-code-review] chan ooh323: IPTOS MINCOST is not defined on Solaris. (asterisk[master])

[Jenkins2]

Jenkins2 has submitted this change and it was merged. ( https://gerrit.asterisk.org/9286 )

Change subject: chan_ooh323: IPTOS_MINCOST is not defined on Solaris.
......................................................................

chan_ooh323: IPTOS_MINCOST is not defined on Solaris.

Furthermore, <sys/sockio.h> is required for SIOCGIF*.

ASTERISK-27938

Change-Id: Idc9153ece769944765b66122efb11728d8d8ebde
---
M addons/chan_ooh323.c
M addons/ooh323c/src/ooSocket.c
2 files changed, 8 insertions(+), 0 deletions(-)

Approvals:
  Joshua Colp: Looks good to me, but someone else must approve
  Benjamin Keith Ford: Looks good to me, but someone else must approve
  George Joseph: Looks good to me, approved
  Jenkins2: Approved for Submit



diff --git a/addons/chan_ooh323.c b/addons/chan_ooh323.c
index 4ab90be..6a65b5f 100644
--- a/addons/chan_ooh323.c
+++ b/addons/chan_ooh323.c
@@ -73,6 +73,10 @@
 #include "chan_ooh323.h"
 #include <math.h>
 
+#ifndef IPTOS_MINCOST
+#define IPTOS_MINCOST	0x02
+#endif
+
 #define FORMAT_STRING_SIZE	512
 
 /* Defaults */
diff --git a/addons/ooh323c/src/ooSocket.c b/addons/ooh323c/src/ooSocket.c
index 1a6a7db..55bb0ac 100644
--- a/addons/ooh323c/src/ooSocket.c
+++ b/addons/ooh323c/src/ooSocket.c
@@ -15,6 +15,10 @@
  *****************************************************************************/
 #include "asterisk.h"
 
+#ifdef SOLARIS
+#include <sys/sockio.h>
+#endif
+
 #include "asterisk/io.h"
 #include "asterisk/lock.h"
 #include "asterisk/utils.h"

-- 
To view, visit https://gerrit.asterisk.org/9286
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: Idc9153ece769944765b66122efb11728d8d8ebde
Gerrit-Change-Number: 9286
Gerrit-PatchSet: 3
Gerrit-Owner: Alexander Traud <pabstraud at compuserve.com>
Gerrit-Reviewer: Benjamin Keith Ford <bford at digium.com>
Gerrit-Reviewer: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: Jenkins2
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20180710/78cedf00/attachment.html>