summaryrefslogtreecommitdiffstats
path: root/src/nlmsg.c
Commit message (Collapse)AuthorAgeFilesLines
* doxygen: fix a variable name.Eric Leblond2013-03-141-1/+1
| | | | | | Variable name in doxygen description was not correct. Signed-off-by: Eric Leblond <eric@regit.org>
* nlmsg: fix valgrind warnings about paddingStephen Hemminger2012-04-191-2/+3
| | | | | | | | | When using mnl_nlmsg_put_extra_header() it pads out the addtional header but only zeros the original size not the padded value. Which cause valgrind to complain about sendto() with uninitialized byte. Signed-off-by: Stephen Hemminger <shemminger@vyatta.com> Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
* doc: fix wrong comment describing mnl_nlmsg_batch_stop()Sean Robinson2011-03-171-3/+3
| | | | | | | | | | | Sean says: "I did find a discrepancy in the comments versus practice in one function and I have attached a patch fixing those comments to this message." Sligtly mangled by myself. Signed-off-by: Sean Robinson <seankrobinson@gmail.com> Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
* doc: don't show up EXPORT_SYMBOL in doxygenPablo Neira Ayuso2011-02-031-28/+40
| | | | | | | | | | | | | | | | | | | | | | | | | | Patrick reports that the doxygen output shows up EXPORT_SYMBOL and tells how to fix this: > I just noticed the EXPORT_SYMBOLs in libmnl showing up in > the doxygen output. Just in case you want to avoid this, > what I'm doing in libdect is (besides the appopriate linker > flags): > > #define __visible __attribute__((visibility("default"))) > > #define EXPORT_SYMBOL(x) typeof(x) (x) __visible > > > This allows to use EXPORT_SYMBOL as in the kernel, IOW > after the function definition. This patch also removes -Wredundant-decls to avoid a warning in every single use of this new approach. Now, this looks more like the Linux kernel way of exporting symbols. Reported-by: Patrick McHardy <kaber@trash.net> Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
* nlmsg: add new message batching infrastructurePablo Neira Ayuso2010-12-171-0/+185
| | | | | | | | | This patch adds the new message batching infrastructure that allows to store several messages into one single datagram. This patch includes an example for ctnetlink. Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
* nlmsg: remove unused function mnl_nlmsg_aligned_size()Jan Engelhardt2010-11-191-12/+0
| | | | | | | | | | | | | With the new CFLAGS, a notification pops up: nlmsg.c:64:8: warning: no previous prototype for "mnl_nlmsg_aligned_size" Marking it thus as static reveals it can be removed. nlmsg.c:64:15: warning: "mnl_nlmsg_aligned_size" defined but not used Signed-off-by: Jan Engelhardt <jengelh@medozas.de> Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
* build: tag function headers rather than decls as exportedJan Engelhardt2010-11-191-13/+20
| | | | Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
* license: change licensing terms from GPLv2+ to LGPLv2.1+Pablo Neira Ayuso2010-11-191-2/+2
| | | | | | | | | | Existing contributors ACK'ed the license change via email: * Jozsef Kadlecsik * Jan Engelhardt * Cristian Rodríguez Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
* nlmsg: use bool for mnl_nlmsg_ok()Jan Engelhardt2010-11-111-1/+1
| | | | | Signed-off-by: Jan Engelhardt <jengelh@medozas.de> Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
* nlmsg: rework mnl_nlmsg_fprintfPablo Neira Ayuso2010-11-071-24/+132
| | | | | | | | This patch reworks mnl_nlmsg_fprintf. It breaks backward compatibility of this function, there was no way to improve it without doing so (and we are still in time to break thing, BTW). Signed-off-bu: Pablo Neira Ayuso <pablo@netfilter.org>
* nlmsg: use bool return type for yes-no functionsJan Engelhardt2010-10-251-9/+9
| | | | | Signed-off-by: Jan Engelhardt <jengelh@medozas.de> Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
* doc: documentation updatesJan Engelhardt2010-10-221-4/+4
| | | | | | Spelling, grammer, and synchronization of the readme. Signed-off-by: Jan Engelhardt <jengelh@medozas.de>
* build: resolve compiler warningsJan Engelhardt2010-10-191-1/+1
| | | | | | | | nlmsg.c: In function "mnl_nlmsg_fprintf": nlmsg.c:260:4: warning: format "%.3d" expects type "int", but argument 3 has type "size_t" Signed-off-by: Jan Engelhardt <jengelh@medozas.de>
* doxygen documentationPablo Neira Ayuso2010-09-081-37/+45
| | | | Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
* skip PortID and sequence checking if zeroPablo Neira Ayuso2010-09-081-4/+13
| | | | | | | | | If the portID/sequence number that we specify is zero, we skip the sequence tracking. This is useful if we use the same socket to listen to events and to send commands and receive their result. Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
* nlmsg: use size_t instead of int for several input parametersPablo Neira Ayuso2010-05-171-4/+4
| | | | Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
* remove redudant alignment in mnl_nlmsg_size()Pablo Neira Ayuso2010-04-221-1/+1
| | | | | | MNL_NLMSG_HDRLEN already provides the aligned size of the netlink header. Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
* add mnl_nlmsg_fprintf() function for debugging purposesPablo Neira Ayuso2010-04-191-11/+17
| | | | | | This function is still quite preliminary, comments welcome! Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
* remove mnl_nlmsg_get_len() functionPablo Neira Ayuso2010-04-121-16/+1
| | | | | | | | | Remove mnl_nlmsg_get_len() since it returns a field of a structure that is public (struct nlmsghdr). We can directly access the header fields and they are not likely to change in the future (at least for this version of Netlink I think). Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
* remove bogus casting in mnl_nlmsg_get_payload_tail()Pablo Neira Ayuso2010-04-121-1/+1
| | | | Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
* improve documentation of netlink message helpersPablo Neira Ayuso2010-04-121-21/+55
| | | | Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
* more consistency name issues: rename get_data*() to get_payload*()Pablo Neira Ayuso2010-04-051-6/+6
| | | | | | This includes renaming get_tail() to get_payload_tail() Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
* rename mnl_nlmsg_payload_size() to mnl_nlmsg_get_payload_len() for consistencyPablo Neira Ayuso2010-04-051-2/+2
| | | | Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
* rename msg.c to nlmsg.cPablo Neira Ayuso2010-04-051-0/+230
Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>