[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20150604152426.GH4549@localhost.localdomain>
Date: Thu, 4 Jun 2015 18:24:26 +0300
From: Sergei Zviagintsev <sergei@...v.net>
To: David Herrmann <dh.herrmann@...il.com>
Cc: Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
Daniel Mack <daniel@...que.org>,
David Herrmann <dh.herrmann@...glemail.com>,
Djalal Harouni <tixxdz@...ndz.org>,
linux-kernel <linux-kernel@...r.kernel.org>, sergei@...v.net
Subject: Re: [PATCH] kdbus: fix type conversion in item macroses
Hi,
On Thu, Jun 04, 2015 at 01:26:03PM +0200, David Herrmann wrote:
> Hi
>
> On Thu, Jun 4, 2015 at 12:39 PM, Sergei Zviagintsev <sergei@...v.net> wrote:
> > KDBUS_ITEM_NEXT and KDBUS_ITEMS_END macroses do not enclose _i argument
> > into parentheses when applying the cast operator. If complex expression
> > passed, e.g.
> >
> > KDBUS_ITEM_NEXT(condition ? a : b)
> >
> > then type conversion to (u8 *) will go wrong due to precedence rules.
> >
> > Use parentheses properly. Update documentation.
> >
> > Signed-off-by: Sergei Zviagintsev <sergei@...v.net>
> > ---
> > Documentation/kdbus/kdbus.item.xml | 2 +-
> > ipc/kdbus/item.h | 4 ++--
> > 2 files changed, 3 insertions(+), 3 deletions(-)
>
> Reviewed-by: David Herrmann <dh.herrmann@...il.com>
It seems that I missed the same issue with KDBUS_ITEMS_SIZE (and
samples/kdbus as well, but that would be another patch), so I'd better
fix that and merge with updated version of
http://permalink.gmane.org/gmane.linux.kernel/1969533
into single patch.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists