From owner-freebsd-bugs@FreeBSD.ORG Thu Sep 4 18:20:03 2008 Return-Path: Delivered-To: freebsd-bugs@hub.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 228741065673 for ; Thu, 4 Sep 2008 18:20:03 +0000 (UTC) (envelope-from gnats@FreeBSD.org) Received: from freefall.freebsd.org (freefall.freebsd.org [IPv6:2001:4f8:fff6::28]) by mx1.freebsd.org (Postfix) with ESMTP id 0FD628FC12 for ; Thu, 4 Sep 2008 18:20:03 +0000 (UTC) (envelope-from gnats@FreeBSD.org) Received: from freefall.freebsd.org (gnats@localhost [127.0.0.1]) by freefall.freebsd.org (8.14.2/8.14.2) with ESMTP id m84IK2GM036788 for ; Thu, 4 Sep 2008 18:20:02 GMT (envelope-from gnats@freefall.freebsd.org) Received: (from gnats@localhost) by freefall.freebsd.org (8.14.2/8.14.1/Submit) id m84IK2LR036787; Thu, 4 Sep 2008 18:20:02 GMT (envelope-from gnats) Date: Thu, 4 Sep 2008 18:20:02 GMT Message-Id: <200809041820.m84IK2LR036787@freefall.freebsd.org> To: freebsd-bugs@FreeBSD.org From: Bruce Cran Cc: Subject: Re: bin/71667: [PATCH] cleanup of the usr.sbin/bootparamd code X-BeenThere: freebsd-bugs@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list Reply-To: Bruce Cran List-Id: Bug reports List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 04 Sep 2008 18:20:03 -0000 The following reply was made to PR bin/71667; it has been noted by GNATS. From: Bruce Cran To: bug-followup@FreeBSD.org, dan@obluda.cz Cc: Subject: Re: bin/71667: [PATCH] cleanup of the usr.sbin/bootparamd code Date: Thu, 4 Sep 2008 19:10:13 +0100 --MP_/dwsL/WnOOCR.BtgJirF=O2V Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Content-Disposition: inline The bootparamd code needs quite a few more changes to make it compile with WARNS=6 with GCC 4. I've attached a patch for callbootd/callbootd.c, but changes are also needed for bootparamd/bootparamd.c -- Bruce Cran --MP_/dwsL/WnOOCR.BtgJirF=O2V Content-Type: text/x-patch; name=callbootd.c.diff Content-Transfer-Encoding: 7bit Content-Disposition: attachment; filename=callbootd.c.diff --- callbootd.c.orig 2008-09-04 18:48:40.000000000 +0100 +++ callbootd.c 2008-09-04 18:57:11.000000000 +0100 @@ -32,15 +32,14 @@ char cln[MAX_MACHINE_NAME+1]; char dmn[MAX_MACHINE_NAME+1]; char path[MAX_PATH_LEN+1]; -extern char *inet_ntoa(); static void usage(void); int printgetfile(bp_getfile_res *); int printwhoami(bp_whoami_res *); +bool_t eachres_whoami(bp_whoami_res *, struct sockaddr_in *); +bool_t eachres_getfile(bp_getfile_res *, struct sockaddr_in *); bool_t -eachres_whoami(resultp, raddr) -bp_whoami_res *resultp; -struct sockaddr_in *raddr; +eachres_whoami(bp_whoami_res *resultp, struct sockaddr_in *raddr) { struct hostent *he; @@ -52,9 +51,7 @@ } bool_t -eachres_getfile(resultp, raddr) -bp_getfile_res *resultp; -struct sockaddr_in *raddr; +eachres_getfile(bp_getfile_res *resultp, struct sockaddr_in *raddr) { struct hostent *he; @@ -67,9 +64,7 @@ int -main(argc, argv) -int argc; -char **argv; +main(int argc, char **argv) { char *server; @@ -80,7 +75,7 @@ long the_inet_addr; - CLIENT *clnt; + CLIENT *clnt = NULL; enum clnt_stat clnt_stat; stat_whoami_res.client_name = cln; @@ -155,6 +150,7 @@ usage(); } + return 0; } --MP_/dwsL/WnOOCR.BtgJirF=O2V--