Skip to content

mdnsd: fix reflector rule acceptance#50

Open
mischief wants to merge 1 commit intohaesbaert:masterfrom
mischief:accept
Open

mdnsd: fix reflector rule acceptance#50
mischief wants to merge 1 commit intohaesbaert:masterfrom
mischief:accept

Conversation

@mischief
Copy link
Copy Markdown
Contributor

c1e71dd (Initial attempt at a reflector., 2021-01-11) added reflector rule parsing, but didn't initialize the rule with the parsed accept/deny, leaving them to always be deny (i think).

assign the parsed value to the rule.

fixes the warning:

mdnsd.c:235:13: warning: variable 'accept' set but not used [-Wunused-but-set-variable]
  235 |         int htype, accept;
      |                    ^

c1e71dd (Initial attempt at a reflector., 2021-01-11) added reflector
rule parsing, but didn't initialize the rule with the parsed
accept/deny, leaving them to always be deny (i think).

assign the parsed value to the rule.

fixes the warning:

	mdnsd.c:235:13: warning: variable 'accept' set but not used [-Wunused-but-set-variable]
	  235 |         int htype, accept;
	      |                    ^
@mischief
Copy link
Copy Markdown
Contributor Author

note i didn't test the reflector/reflector rule parsing at all, this is only compile tested.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant