mtest: Prevent access to uninitialized value.

case 'g' makes use of value n, which is initialized for case 'b'
and passe through to case 'g'. If case 'g' is called directly
value 'n' is not initialized.

Avoid the issue by initializing n before entering the switch.

CID:		1006375
Reviewed by:	bms

This commit is contained in:

Pedro F. Giffuni

2016-04-01 01:35:52 +00:00

parent f4864c274b

commit 2b5554fec0

1 changed files with 1 additions and 1 deletions

									
										usr.sbin/mtest/mtest.c
									
		+1
		-1
	
												View File
												
				@@ -396,6 +396,7 @@ process_cmd(char *cmd, int s, int s6, FILE *fp __unused)

					while (isblank(*++line))

						;	/* Skip whitespace. */

					n = 0;

					switch (*cmd) {

					case '?':

						usage();

				@@ -611,7 +612,6 @@ process_cmd(char *cmd, int s, int s6, FILE *fp __unused)

						}

						/* First determine our current filter mode. */

						n = 0;

						if (getsourcefilter(af2sock(af, s, s6), ifindex,

						    &su.sa, su.sa.sa_len, &fmode, &n, NULL) != 0) {

							warn("getsourcefilter");