Module Name:    src
Committed By:   mrg
Date:           Sun Feb  4 09:01:13 UTC 2018

Modified Files:
        src/usr.bin/mail: lex.c
        src/usr.bin/telnet: sys_bsd.c
        src/usr.sbin/isdn/isdnd: rc_config.c
        src/usr.sbin/isdn/isdntrace: q931_util.c
        src/usr.sbin/map-mbone: mapper.c
        src/usr.sbin/sysinst: partman.c
        src/usr.sbin/timed/timed: timed.c

Log Message:
fixes for GCC 6.4:

mail's execute() needs a volatile for setjmp().

telnet has a missing {} issue.

isdnd's print_config() has a missing/wrong {} issue, and
its p_q931bc() has inconsistent indentation (but not any
actual problem.)

map-mbone's accept_neighbors2() compares a vs. a instead
of a vs. b.

sysinst's pm_cgd_check() has missing {} issue.

timed's main() has missing {} issue.


To generate a diff of this commit:
cvs rdiff -u -r1.44 -r1.45 src/usr.bin/mail/lex.c
cvs rdiff -u -r1.33 -r1.34 src/usr.bin/telnet/sys_bsd.c
cvs rdiff -u -r1.26 -r1.27 src/usr.sbin/isdn/isdnd/rc_config.c
cvs rdiff -u -r1.3 -r1.4 src/usr.sbin/isdn/isdntrace/q931_util.c
cvs rdiff -u -r1.26 -r1.27 src/usr.sbin/map-mbone/mapper.c
cvs rdiff -u -r1.17 -r1.18 src/usr.sbin/sysinst/partman.c
cvs rdiff -u -r1.25 -r1.26 src/usr.sbin/timed/timed/timed.c

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.

Modified files:

Index: src/usr.bin/mail/lex.c
diff -u src/usr.bin/mail/lex.c:1.44 src/usr.bin/mail/lex.c:1.45
--- src/usr.bin/mail/lex.c:1.44	Thu Nov  9 20:27:50 2017
+++ src/usr.bin/mail/lex.c	Sun Feb  4 09:01:12 2018
@@ -1,4 +1,4 @@
-/*	$NetBSD: lex.c,v 1.44 2017/11/09 20:27:50 christos Exp $	*/
+/*	$NetBSD: lex.c,v 1.45 2018/02/04 09:01:12 mrg Exp $	*/
 
 /*
  * Copyright (c) 1980, 1993
@@ -34,7 +34,7 @@
 #if 0
 static char sccsid[] = "@(#)lex.c	8.2 (Berkeley) 4/20/95";
 #else
-__RCSID("$NetBSD: lex.c,v 1.44 2017/11/09 20:27:50 christos Exp $");
+__RCSID("$NetBSD: lex.c,v 1.45 2018/02/04 09:01:12 mrg Exp $");
 #endif
 #endif /* not lint */
 
@@ -627,7 +627,7 @@ execute(char linebuf[], enum execute_con
 	char *volatile cp;
 	int retval;
 	int c;
-	int e = 1;
+	volatile int e = 1;
 
 	/*
 	 * Strip the white space away from the beginning

Index: src/usr.bin/telnet/sys_bsd.c
diff -u src/usr.bin/telnet/sys_bsd.c:1.33 src/usr.bin/telnet/sys_bsd.c:1.34
--- src/usr.bin/telnet/sys_bsd.c:1.33	Mon Jan  9 16:08:55 2012
+++ src/usr.bin/telnet/sys_bsd.c	Sun Feb  4 09:01:12 2018
@@ -1,4 +1,4 @@
-/*	$NetBSD: sys_bsd.c,v 1.33 2012/01/09 16:08:55 christos Exp $	*/
+/*	$NetBSD: sys_bsd.c,v 1.34 2018/02/04 09:01:12 mrg Exp $	*/
 
 /*
  * Copyright (c) 1988, 1990, 1993
@@ -34,7 +34,7 @@
 #if 0
 from: static char sccsid[] = "@(#)sys_bsd.c	8.4 (Berkeley) 5/30/95";
 #else
-__RCSID("$NetBSD: sys_bsd.c,v 1.33 2012/01/09 16:08:55 christos Exp $");
+__RCSID("$NetBSD: sys_bsd.c,v 1.34 2018/02/04 09:01:12 mrg Exp $");
 #endif
 #endif /* not lint */
 
@@ -468,11 +468,12 @@ TerminalSpeeds(long *ispeed, long *ospee
 
     out = cfgetospeed(&old_tc);
     in = cfgetispeed(&old_tc);
-    if (in == 0)
+    if (in == 0) {
 	in = out;
 
 	*ispeed = in;
 	*ospeed = out;
+    }
 }
 
 int

Index: src/usr.sbin/isdn/isdnd/rc_config.c
diff -u src/usr.sbin/isdn/isdnd/rc_config.c:1.26 src/usr.sbin/isdn/isdnd/rc_config.c:1.27
--- src/usr.sbin/isdn/isdnd/rc_config.c:1.26	Thu Mar  1 22:31:48 2012
+++ src/usr.sbin/isdn/isdnd/rc_config.c	Sun Feb  4 09:01:12 2018
@@ -27,7 +27,7 @@
  *	i4b daemon - config file processing
  *	-----------------------------------
  *
- *	$Id: rc_config.c,v 1.26 2012/03/01 22:31:48 joerg Exp $ 
+ *	$Id: rc_config.c,v 1.27 2018/02/04 09:01:12 mrg Exp $ 
  *
  * $FreeBSD$
  *
@@ -1564,20 +1564,19 @@ print_config(void)
 			for (j = 0; j < cep->remote_numbers_count; j++)
 				fprintf(PFILE, "remote-phone-dialout  = %s\t\t# telephone number %d for dialing out to remote\n", cep->remote_numbers[j].number, j+1);
 
-				fprintf(PFILE, "remdial-handling      = ");
+			fprintf(PFILE, "remdial-handling      = ");
 	
-				switch (cep->remote_numbers_handling)
-				{
-				case RNH_NEXT:
-					fprintf(PFILE, "next\t\t# use next number after last successful for new dial\n");
-					break;
-				case RNH_LAST:
-					fprintf(PFILE, "last\t\t# use last successful number for new dial\n");
-					break;
-				case RNH_FIRST:
-					fprintf(PFILE, "first\t\t# always start with first number for new dial\n");
-					break;
-				}
+			switch (cep->remote_numbers_handling)
+			{
+			case RNH_NEXT:
+				fprintf(PFILE, "next\t\t# use next number after last successful for new dial\n");
+				break;
+			case RNH_LAST:
+				fprintf(PFILE, "last\t\t# use last successful number for new dial\n");
+				break;
+			case RNH_FIRST:
+				fprintf(PFILE, "first\t\t# always start with first number for new dial\n");
+				break;
 			}
 
 			if (cep->local_phone_dialout[0])
@@ -1752,6 +1751,7 @@ print_config(void)
 					fprintf(PFILE, "downtime              = %d\t\t# time device is switched off\n", cep->downtime);
 				}
 			}
+		}
 	}
 	fprintf(PFILE, "\n");	
 }

Index: src/usr.sbin/isdn/isdntrace/q931_util.c
diff -u src/usr.sbin/isdn/isdntrace/q931_util.c:1.3 src/usr.sbin/isdn/isdntrace/q931_util.c:1.4
--- src/usr.sbin/isdn/isdntrace/q931_util.c:1.3	Mon Oct  6 09:43:28 2003
+++ src/usr.sbin/isdn/isdntrace/q931_util.c	Sun Feb  4 09:01:12 2018
@@ -27,7 +27,7 @@
  *	q931_util.c - utility functions to print Q.931 traces
  *	-----------------------------------------------------
  *
- *	$Id: q931_util.c,v 1.3 2003/10/06 09:43:28 itojun Exp $ 
+ *	$Id: q931_util.c,v 1.4 2018/02/04 09:01:12 mrg Exp $ 
  *
  * $FreeBSD$
  *
@@ -405,38 +405,38 @@ p_q931bc(char *pbuf, unsigned char *buf)
 	if (!len)
 		goto exit;
 
-		switch (buf[i] & 0x7f)
-		{
-			case 0x42:
-				sprintf((pbuf+strlen(pbuf)), "\n          layer2=Q.921/I.441");
-				break;
-			case 0x46:
-				sprintf((pbuf+strlen(pbuf)), "\n          layer2=X.25 link");
-				break;
-			default:
-				sprintf((pbuf+strlen(pbuf)), "\n          layer2=0x%02x",(buf[i] & 0x7f));
-				break;
-		}
-		i++;
-		len--;
+	switch (buf[i] & 0x7f)
+	{
+		case 0x42:
+			sprintf((pbuf+strlen(pbuf)), "\n          layer2=Q.921/I.441");
+			break;
+		case 0x46:
+			sprintf((pbuf+strlen(pbuf)), "\n          layer2=X.25 link");
+			break;
+		default:
+			sprintf((pbuf+strlen(pbuf)), "\n          layer2=0x%02x",(buf[i] & 0x7f));
+			break;
+	}
+	i++;
+	len--;
 
 	if (!len)
 		goto exit;
 	
-		switch (buf[i] & 0x7f)
-		{
-			case 0x62:
-				sprintf((pbuf+strlen(pbuf)), "\n          layer3=Q.921/I.441");
-				break;
-			case 0x66:
-				sprintf((pbuf+strlen(pbuf)), "\n          layer3=X.25 packet");
-				break;
-			default:
-				sprintf((pbuf+strlen(pbuf)), "\n          layer3=0x%02x",(buf[i] & 0x7f));
-				break;
-		}
-		i++;
-		len--;
+	switch (buf[i] & 0x7f)
+	{
+		case 0x62:
+			sprintf((pbuf+strlen(pbuf)), "\n          layer3=Q.921/I.441");
+			break;
+		case 0x66:
+			sprintf((pbuf+strlen(pbuf)), "\n          layer3=X.25 packet");
+			break;
+		default:
+			sprintf((pbuf+strlen(pbuf)), "\n          layer3=0x%02x",(buf[i] & 0x7f));
+			break;
+	}
+	i++;
+	len--;
 
 exit:	
 	sprintf((pbuf+strlen(pbuf)), "]");	

Index: src/usr.sbin/map-mbone/mapper.c
diff -u src/usr.sbin/map-mbone/mapper.c:1.26 src/usr.sbin/map-mbone/mapper.c:1.27
--- src/usr.sbin/map-mbone/mapper.c:1.26	Tue Jan 23 23:02:40 2018
+++ src/usr.sbin/map-mbone/mapper.c	Sun Feb  4 09:01:13 2018
@@ -1,4 +1,4 @@
-/*	$NetBSD: mapper.c,v 1.26 2018/01/23 23:02:40 sevan Exp $	*/
+/*	$NetBSD: mapper.c,v 1.27 2018/02/04 09:01:13 mrg Exp $	*/
 
 /* Mapper for connections between MRouteD multicast routers.
  * Written by Pavel Curtis <pa...@parc.xerox.com>
@@ -495,7 +495,7 @@ void accept_neighbors2(u_int32_t src, u_
 		    for (nb_n = old_neighbors; nb_n; nb_n = nb_n->next)
 			if (nb_i->addr == nb_n->addr) {
 			    if (nb_i->metric != nb_n->metric
-				|| nb_i->threshold != nb_i->threshold)
+				|| nb_i->threshold != nb_n->threshold)
 				logit(LOG_WARNING, 0,
 				    "inconsistent %s for neighbor %s of %s",
 				    "metric/threshold",

Index: src/usr.sbin/sysinst/partman.c
diff -u src/usr.sbin/sysinst/partman.c:1.17 src/usr.sbin/sysinst/partman.c:1.18
--- src/usr.sbin/sysinst/partman.c:1.17	Mon Jan 29 16:24:23 2018
+++ src/usr.sbin/sysinst/partman.c	Sun Feb  4 09:01:13 2018
@@ -1,4 +1,4 @@
-/*	$NetBSD: partman.c,v 1.17 2018/01/29 16:24:23 martin Exp $ */
+/*	$NetBSD: partman.c,v 1.18 2018/02/04 09:01:13 mrg Exp $ */
 
 /*
  * Copyright 2012 Eugene Lozovoy
@@ -1099,10 +1099,11 @@ pm_cgd_check(void *arg)
 		return 0;
 	if (dev_ptr->pm == NULL)
 		dev_ptr->enabled = 0;
-	else
+	else {
 		pm_manage_getfreenode(&(dev_ptr->node), "cgd", &cgds_t_info);
 		if (dev_ptr->node < 0)
 			dev_ptr->enabled = 0;
+	}
 	return dev_ptr->enabled;
 }
 

Index: src/usr.sbin/timed/timed/timed.c
diff -u src/usr.sbin/timed/timed/timed.c:1.25 src/usr.sbin/timed/timed/timed.c:1.26
--- src/usr.sbin/timed/timed/timed.c:1.25	Tue Jun 16 23:04:14 2015
+++ src/usr.sbin/timed/timed/timed.c	Sun Feb  4 09:01:13 2018
@@ -1,4 +1,4 @@
-/*	$NetBSD: timed.c,v 1.25 2015/06/16 23:04:14 christos Exp $	*/
+/*	$NetBSD: timed.c,v 1.26 2018/02/04 09:01:13 mrg Exp $	*/
 
 /*-
  * Copyright (c) 1985, 1993 The Regents of the University of California.
@@ -39,7 +39,7 @@ __COPYRIGHT("@(#) Copyright (c) 1985, 19
 #if 0
 static char sccsid[] = "@(#)timed.c	8.2 (Berkeley) 3/26/95";
 #else
-__RCSID("$NetBSD: timed.c,v 1.25 2015/06/16 23:04:14 christos Exp $");
+__RCSID("$NetBSD: timed.c,v 1.26 2018/02/04 09:01:13 mrg Exp $");
 #endif
 #endif /* not lint */
 
@@ -405,9 +405,10 @@ main(int argc, char *argv[])
 			justquit = 1;
 		}
 		for (ntp = nettab; ntp != NULL; ntp = ntp->next) {
-			if (ntp->status == MASTER)
+			if (ntp->status == MASTER) {
 				rmnetmachs(ntp);
 				ntp->status = NOMASTER;
+			}
 		}
 		checkignorednets();
 		pickslavenet(0);

Reply via email to