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

[Alexander Traud]

Alexander Traud has uploaded this change for review. ( 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(-)



  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/86/9286/2

diff --git a/addons/chan_ooh323.c b/addons/chan_ooh323.c
index 4ab90be..8737fec 100644
--- a/addons/chan_ooh323.c
+++ b/addons/chan_ooh323.c
@@ -3050,7 +3050,11 @@
 			else if (!strcasecmp(v->value, "reliability"))
 				gTOS = IPTOS_RELIABILITY;
 			else if (!strcasecmp(v->value, "mincost"))
+#ifdef IPTOS_MINCOST
 				gTOS = IPTOS_MINCOST;
+#else
+				gTOS = 0x02;
+#endif
 			else if (!strcasecmp(v->value, "none"))
 				gTOS = 0;
 			else
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, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Idc9153ece769944765b66122efb11728d8d8ebde
Gerrit-Change-Number: 9286
Gerrit-PatchSet: 2
Gerrit-Owner: Alexander Traud <pabstraud at compuserve.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20180623/b8821e1e/attachment.html>