From 3627fbc9294961ff679411cf205da39b09b366f7 Mon Sep 17 00:00:00 2001
From: bodea <bodea>
Date: Mon, 15 Nov 2004 02:26:20 +0000
Subject: [PATCH] =?utf8?q?Fix=20socket=20creation=20in=20host=5Funreachabl?=
 =?utf8?q?e()=20(thanks=20to=20Bj=F8rn=20Augestad)?=
MIME-Version: 1.0
Content-Type: text/plain; charset=utf8
Content-Transfer-Encoding: 8bit

---
 Changes | 3 +++
 icmp.c  | 5 +++--
 2 files changed, 6 insertions(+), 2 deletions(-)

diff --git a/Changes b/Changes
index 0d6ee8d..5ac335f 100644
--- a/Changes
+++ b/Changes
@@ -1,3 +1,6 @@
+* ? Brendan O'Dea <bod@optusnet.com.au> 2.0.7
+- Fix socket creation in host_unreachable() (thanks to Bjørn Augestad)
+
 * Thu Nov 11 2004 Brendan O'Dea <bod@optusnet.com.au> 2.0.6
 - Make BGP keepalive/hold time configurable
 - Revise BGP config to use "router bgp AS" syntax (requires libcli >= 1.8.2)
diff --git a/icmp.c b/icmp.c
index 4775d49..d115122 100644
--- a/icmp.c
+++ b/icmp.c
@@ -1,6 +1,6 @@
 // L2TPNS: icmp
 
-char const *cvs_id_icmp = "$Id: icmp.c,v 1.3 2004/06/28 02:43:13 fred_nerk Exp $";
+char const *cvs_id_icmp = "$Id: icmp.c,v 1.4 2004/11/15 02:26:20 bodea Exp $";
 
 #include <arpa/inet.h>
 #include <netdb.h>
@@ -28,8 +28,9 @@ void host_unreachable(ipt destination, u16 id, ipt source, char *packet, int pac
 	int len = 0, on = 1, icmp_socket;
 	struct sockaddr_in whereto = {0};
 
-	if (!(icmp_socket = socket(AF_INET, SOCK_RAW, IPPROTO_RAW)))
+	if ((icmp_socket = socket(AF_INET, SOCK_RAW, IPPROTO_RAW)) < 0)
 		return;
+
 	setsockopt(icmp_socket, IPPROTO_IP, IP_HDRINCL, (char *)&on, sizeof(on));
 
 	whereto.sin_addr.s_addr = destination;
-- 
2.20.1