On Wed, Jun 15, 2016 at 12:43:04PM +0000, Fabien Siron wrote:
> Since our target is not in the same process, the netlink macros NLMSG_*
> must be rewritten for strace.
>
> * netlink.c (nlmsg_ok, nlmsg_next): New functions.
> ---
> netlink.c | 26 ++++++++++++++++++++++++++
> 1 file changed, 26 insertions(+)
>
> diff --git a/netlink.c b/netlink.c
> index 30116dd..c1a9ad4 100644
> --- a/netlink.c
> +++ b/netlink.c
> @@ -31,6 +31,32 @@
> #include "xlat/netlink_flags.h"
> #include "xlat/netlink_types.h"
>
> +/* since our target is not in the same process, here are some utils for
> nlmsg */
> +static int
> +nlmsg_ok(struct tcb *tcp, unsigned long nlh, unsigned long len) {
> + struct nlmsghdr nlmsghdr;
> +
> + if (umoven(tcp, nlh, sizeof(struct nlmsghdr), &nlmsghdr) == -1)
> + return 0;
> +
> + return (len >= (int)sizeof(struct nlmsghdr)) &&
Shouldn't umoven be skipped if this condition is false?
> + nlmsghdr.nlmsg_len >= sizeof(struct nlmsghdr) &&
> + nlmsghdr.nlmsg_len <= len;
> +}
> +
> +static unsigned long
> +nlmsg_next(struct tcb *tcp, unsigned long nlh, unsigned long *len) {
> + struct nlmsghdr nlmsghdr;
> +
> + if (umoven(tcp, nlh, sizeof(struct nlmsghdr), &nlmsghdr) == -1)
> + return 0;
> +
> + *len -= NLMSG_ALIGN(nlmsghdr.nlmsg_len);
> +
> + return (unsigned long)
> + ((char *)(nlh) + NLMSG_ALIGN(nlmsghdr.nlmsg_len));
> +}
> +
> void
> decode_netlink(struct tcb *tcp, unsigned long addr, unsigned long size)
> {
All these three functions (nlmsg_ok, nlmsg_next, and decode_netlink) fetch
struct nlmsghdr, and likely they fetch the same structure. Couldn't this
inefficiency be avoided?
--
ldv
pgpRKWWKuiacx.pgp
Description: PGP signature
------------------------------------------------------------------------------ What NetFlow Analyzer can do for you? Monitors network bandwidth and traffic patterns at an interface-level. Reveals which users, apps, and protocols are consuming the most bandwidth. Provides multi-vendor support for NetFlow, J-Flow, sFlow and other flows. Make informed decisions using capacity planning reports. http://sdm.link/zohomanageengine
_______________________________________________ Strace-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/strace-devel
