netlink: abolish check_config and output_config
authorIan Jackson <ijackson@chiark.greenend.org.uk>
Wed, 13 Jun 2012 20:06:02 +0000 (21:06 +0100)
committerIan Jackson <ijackson@chiark.greenend.org.uk>
Thu, 12 Jul 2012 19:02:21 +0000 (20:02 +0100)
commitff7cdc9ea887dfef812d914ba4f0a3e62f4c0e58
tree412331069224b310a6c29d9da4eff3638b35b8b7
parentd714da29de327460f40d996e9cf69b71ccdcaa31
netlink: abolish check_config and output_config

Apparently, a long time ago, MSG5 and MSG6 used to contain some
netlink configuration data, which the receiver of the MSG5 or MSG6
would check.

However, for a long time now the output_config function has been a
no-op and the check function has unconditionally eaten and discarded
anything extra in the message.

Furthermore, because the MSG6 is not retransmitted, this mechanism
couldn't be reliable without a protocol change.  So the existing
interface is defective.

So, abolish it the interface, the dummy implementation, and all the
call sites.  The check_config call sites in site.c now instead
directly discard any unexpected data at the end of MSG5 and MSG6.

This patch should cause no behavioural change in actual operation.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
netlink.c
secnet.h
site.c