From owner-svn-src-user@freebsd.org  Wed May 18 22:49:32 2016
Return-Path: <owner-svn-src-user@freebsd.org>
Delivered-To: svn-src-user@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 06679B4101F
 for <svn-src-user@mailman.ysv.freebsd.org>;
 Wed, 18 May 2016 22:49:32 +0000 (UTC)
 (envelope-from glebius@FreeBSD.org)
Received: from cell.glebi.us (glebi.us [96.95.210.25])
 (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits))
 (Client CN "cell.glebi.us", Issuer "cell.glebi.us" (not verified))
 by mx1.freebsd.org (Postfix) with ESMTPS id E1DE21A2A;
 Wed, 18 May 2016 22:49:31 +0000 (UTC)
 (envelope-from glebius@FreeBSD.org)
Received: from cell.glebi.us (localhost [127.0.0.1])
 by cell.glebi.us (8.15.2/8.15.2) with ESMTPS id u4IMnOOf047359
 (version=TLSv1.2 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=NO);
 Wed, 18 May 2016 15:49:25 -0700 (PDT)
 (envelope-from glebius@FreeBSD.org)
Received: (from glebius@localhost)
 by cell.glebi.us (8.15.2/8.15.2/Submit) id u4IMnOhf047358;
 Wed, 18 May 2016 15:49:24 -0700 (PDT)
 (envelope-from glebius@FreeBSD.org)
X-Authentication-Warning: cell.glebi.us: glebius set sender to
 glebius@FreeBSD.org using -f
Date: Wed, 18 May 2016 15:49:24 -0700
From: Gleb Smirnoff <glebius@FreeBSD.org>
To: Garrett Cooper <ngie@FreeBSD.org>
Cc: src-committers@freebsd.org, svn-src-user@freebsd.org
Subject: Re: svn commit: r299818 - user/ngie/bsnmp_cleanup/contrib/bsnmp/snmpd
Message-ID: <20160518224924.GM1015@FreeBSD.org>
References: <201605150110.u4F1AW92016063@repo.freebsd.org>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <201605150110.u4F1AW92016063@repo.freebsd.org>
User-Agent: Mutt/1.6.1 (2016-04-27)
X-BeenThere: svn-src-user@freebsd.org
X-Mailman-Version: 2.1.22
Precedence: list
List-Id: "SVN commit messages for the experimental &quot; user&quot;
 src tree" <svn-src-user.freebsd.org>
List-Unsubscribe: <https://lists.freebsd.org/mailman/options/svn-src-user>,
 <mailto:svn-src-user-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/svn-src-user/>
List-Post: <mailto:svn-src-user@freebsd.org>
List-Help: <mailto:svn-src-user-request@freebsd.org?subject=help>
List-Subscribe: <https://lists.freebsd.org/mailman/listinfo/svn-src-user>,
 <mailto:svn-src-user-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Wed, 18 May 2016 22:49:32 -0000

On Sun, May 15, 2016 at 01:10:32AM +0000, Garrett Cooper wrote:
G> Author: ngie
G> Date: Sun May 15 01:10:31 2016
G> New Revision: 299818
G> URL: https://svnweb.freebsd.org/changeset/base/299818
G> 
G> Log:
G>   Test laddr.s_addr against NULL, not 0
G> 
G> Modified:
G>   user/ngie/bsnmp_cleanup/contrib/bsnmp/snmpd/main.c
G> 
G> Modified: user/ngie/bsnmp_cleanup/contrib/bsnmp/snmpd/main.c
G> ==============================================================================
G> --- user/ngie/bsnmp_cleanup/contrib/bsnmp/snmpd/main.c	Sun May 15 00:54:21 2016	(r299817)
G> +++ user/ngie/bsnmp_cleanup/contrib/bsnmp/snmpd/main.c	Sun May 15 01:10:31 2016	(r299818)
G> @@ -1220,7 +1220,7 @@ snmpd_input(struct port_input *pi, struc
G>  		
G>  		ret = recv_dgram(pi, &laddr);
G>  
G> -		if (laddr.s_addr == 0) {
G> +		if (laddr.s_addr == NULL) {
G>  			msg.msg_control = NULL;
G>  			msg.msg_controllen = 0;
G>  		}

This change also seems incorrect. It isn't a pointer.

-- 
Totus tuus, Glebius.