diff options
| author | Thomas Egerer <hakke_007@gmx.de> | 2015-06-04 12:43:58 (GMT) |
|---|---|---|
| committer | Thomas Haller <thaller@redhat.com> | 2015-06-05 06:40:02 (GMT) |
| commit | bbdcaea9a779885fedc04817dcc11953a377bfd5 (patch) | |
| tree | a498d229e236c8fd01b2be85fbd88deef3e7c4d4 /lib | |
| parent | e206c255d8fff8b51938945c8f575750c418a95e (diff) | |
| download | libnl-bbdcaea9a779885fedc04817dcc11953a377bfd5.zip libnl-bbdcaea9a779885fedc04817dcc11953a377bfd5.tar.gz libnl-bbdcaea9a779885fedc04817dcc11953a377bfd5.tar.bz2 | |
lib: return error if an incomplete message was read
If recvmsg indicates that the message read was truncated libnl retries
to read the complete message after increasing the message buffer. This
only works if the message flags MSG_PEEK | MSG_TRUNC are set. If
NL_MSG_PEEK is not enabled on the nl_sock structure, flags are left
empty and the rest of the truncated message is discarded, hence a
subsequent recvmsg returns the next message (in case of a multipart
message, the NLMSG_DONE) is read and returned.
This patch aborts message processing if the message was truncated and
the NL_MSG_PEEK flags was not activated for the nl_sock structure.
http://lists.infradead.org/pipermail/libnl/2015-June/001888.html
[thaller@redhat.com: add NL_CAPABILITY_NL_RECV_FAIL_TRUNK_NO_PEEK]
Signed-off-by: Thomas Egerer <hakke_007@gmx.de>
Signed-off-by: Thomas Haller <thaller@redhat.com>
Diffstat (limited to 'lib')
| -rw-r--r-- | lib/nl.c | 7 | ||||
| -rw-r--r-- | lib/utils.c | 2 |
2 files changed, 8 insertions, 1 deletions
@@ -725,6 +725,13 @@ retry: if (iov.iov_len < n || (msg.msg_flags & MSG_TRUNC)) { void *tmp; + + /* respond with error to an incomplete message */ + if (!(sk->s_flags & NL_MSG_PEEK)) { + retval = -NLE_MSG_TRUNC; + goto abort; + } + /* Provided buffer is not long enough, enlarge it * to size of n (which should be total length of the message) * and try again. */ diff --git a/lib/utils.c b/lib/utils.c index 82f14ac..aa0d1e1 100644 --- a/lib/utils.c +++ b/lib/utils.c @@ -1150,7 +1150,7 @@ int nl_has_capability (int capability) NL_CAPABILITY_ROUTE_LINK_GET_KERNEL_FAIL_OPNOTSUPP, NL_CAPABILITY_ROUTE_ADDR_COMPARE_CACHEINFO, NL_CAPABILITY_VERSION_3_2_26, - 0), + NL_CAPABILITY_NL_RECV_FAIL_TRUNK_NO_PEEK), /* IMPORTANT: these capability numbers are intended to be universal and stable * for libnl3. Don't allocate new numbers on your own that differ from upstream * libnl3. |