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

[Alexander Traud]

Alexander Traud has uploaded this change for review. ( https://gerrit.asterisk.org/9287


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/87/9287/1

diff --git a/addons/chan_ooh323.c b/addons/chan_ooh323.c
index ac6d808..79e9362 100644
--- a/addons/chan_ooh323.c
+++ b/addons/chan_ooh323.c
@@ -3045,7 +3045,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/9287
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: 15
Gerrit-MessageType: newchange
Gerrit-Change-Id: Idc9153ece769944765b66122efb11728d8d8ebde
Gerrit-Change-Number: 9287
Gerrit-PatchSet: 1
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/5ccbdc47/attachment.html>