From mboxrd@z Thu Jan 1 00:00:00 1970 From: Scott Feldman Subject: Re: Problem with patch "make nlmsg_end() and genlmsg_end() void" Date: Mon, 19 Jan 2015 01:31:24 -0800 Message-ID: References: <01A82AB9-6ABD-4AD0-9CBC-628091569DB0@holtmann.org> <20150118.233722.226468667930444145.davem@davemloft.net> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Cc: Marcel Holtmann , Netdev , Johannes Berg , Tom Gundersen To: David Miller Return-path: Received: from mail-wg0-f46.google.com ([74.125.82.46]:33852 "EHLO mail-wg0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751435AbbASJbZ (ORCPT ); Mon, 19 Jan 2015 04:31:25 -0500 Received: by mail-wg0-f46.google.com with SMTP id x13so30606473wgg.5 for ; Mon, 19 Jan 2015 01:31:24 -0800 (PST) In-Reply-To: <20150118.233722.226468667930444145.davem@davemloft.net> Sender: netdev-owner@vger.kernel.org List-ID: On Sun, Jan 18, 2015 at 8:37 PM, David Miller wrote: > > ==================== > [PATCH] netlink: Fix bugs in nlmsg_end() conversions. > > Commit 053c095a82cf ("netlink: make nlmsg_end() and genlmsg_end() > void") didn't catch all of the cases where callers were breaking out > on the return value being equal to zero, which they no longer should > when zero means success. > > Fix all such cases. > > Reported-by: Marcel Holtmann > Reported-by: Scott Feldman > Signed-off-by: David S. Miller Fixes my tests, so I'm happy. Thank you. Acked-by: Scott Feldman