From owner-freebsd-net@freebsd.org  Tue Mar 15 02:05:11 2016
Return-Path: <owner-freebsd-net@freebsd.org>
Delivered-To: freebsd-net@mailman.ysv.freebsd.org
Received: from mx1.freebsd.org (mx1.freebsd.org
 [IPv6:2001:1900:2254:206a::19:1])
 by mailman.ysv.freebsd.org (Postfix) with ESMTP id D67ECACF134
 for <freebsd-net@mailman.ysv.freebsd.org>;
 Tue, 15 Mar 2016 02:05:11 +0000 (UTC)
 (envelope-from mmcco@mykolab.com)
Received: from mx-out03.mykolab.com (mx01.mykolab.com [95.128.36.1])
 (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits))
 (Client did not present a certificate)
 by mx1.freebsd.org (Postfix) with ESMTPS id A438FBE4
 for <freebsd-net@freebsd.org>; Tue, 15 Mar 2016 02:05:10 +0000 (UTC)
 (envelope-from mmcco@mykolab.com)
X-Virus-Scanned: amavisd-new at kolabnow.com
X-Spam-Flag: NO
X-Spam-Score: -2.9
X-Spam-Level: 
X-Spam-Status: No, score=-2.9 tagged_above=-10 required=6.31
 tests=[ALL_TRUSTED=-1, BAYES_00=-1.9, FREEMAIL_FROM=0.001,
 RP_MATCHES_RCVD=-0.001] autolearn=ham
Received: from mx04.mykolab.com (mx04.mykolab.com [10.20.7.102])
 by mx-out03.mykolab.com (Postfix) with ESMTPS id D76D723EBF
 for <freebsd-net@freebsd.org>; Tue, 15 Mar 2016 02:55:45 +0100 (CET)
Date: Mon, 14 Mar 2016 21:55:42 -0400
From: Michael McConville <mmcco@mykolab.com>
To: freebsd-net@freebsd.org
Subject: Int overflow in dhclient(1)
Message-ID: <20160315015542.GA61626@thinkpad.swarthmore.edu>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
X-BeenThere: freebsd-net@freebsd.org
X-Mailman-Version: 2.1.21
Precedence: list
List-Id: Networking and TCP/IP with FreeBSD <freebsd-net.freebsd.org>
List-Unsubscribe: <https://lists.freebsd.org/mailman/options/freebsd-net>,
 <mailto:freebsd-net-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/freebsd-net/>
List-Post: <mailto:freebsd-net@freebsd.org>
List-Help: <mailto:freebsd-net-request@freebsd.org?subject=help>
List-Subscribe: <https://lists.freebsd.org/mailman/listinfo/freebsd-net>,
 <mailto:freebsd-net-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Tue, 15 Mar 2016 02:05:11 -0000

I sent this a few weeks ago, but it never got applied:

https://marc.info/?l=freebsd-net&m=145392057613453&w=2

Here's my explanation for the same fix in OpenBSD:

https://marc.info/?l=openbsd-tech&m=145377854103866&w=2

Thanks for your time,
Michael


Index: sbin/dhclient/dhclient.c
===================================================================
--- sbin/dhclient/dhclient.c	(revision 296886)
+++ sbin/dhclient/dhclient.c	(working copy)
@@ -138,7 +138,7 @@
 findproto(char *cp, int n)
 {
 	struct sockaddr *sa;
-	int i;
+	unsigned int i;
 
 	if (n == 0)
 		return -1;