From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753299AbbJWML3 (ORCPT ); Fri, 23 Oct 2015 08:11:29 -0400 Received: from mail-wi0-f193.google.com ([209.85.212.193]:33648 "EHLO mail-wi0-f193.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751547AbbJWML1 (ORCPT ); Fri, 23 Oct 2015 08:11:27 -0400 From: LABBE Corentin To: alexander.h.duyck@redhat.com, davem@davemloft.net, edumazet@google.com, fw@strlen.de, hannes@stressinduktion.org, jiri@resnulli.us, pablo@netfilter.org, tom@herbertland.com, viro@zeniv.linux.org.uk Cc: LABBE Corentin , linux-kernel@vger.kernel.org Subject: [PATCH 01/11] net: change len parameter type for memcpy_[to|from]_msg Date: Fri, 23 Oct 2015 14:10:26 +0200 Message-Id: <1445602236-24130-2-git-send-email-clabbe.montjoie@gmail.com> X-Mailer: git-send-email 2.4.10 In-Reply-To: <1445602236-24130-1-git-send-email-clabbe.montjoie@gmail.com> References: <1445602236-24130-1-git-send-email-clabbe.montjoie@gmail.com> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org The len parameter is only used with copy_from_iter() who wait for a size_t and then compared to return type of copy_from_iter() which is size_t also. So len must be set as size_t Signed-off-by: LABBE Corentin --- include/linux/skbuff.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/include/linux/skbuff.h b/include/linux/skbuff.h index 4398411..109a127 100644 --- a/include/linux/skbuff.h +++ b/include/linux/skbuff.h @@ -2827,12 +2827,12 @@ int skb_ensure_writable(struct sk_buff *skb, int write_len); int skb_vlan_pop(struct sk_buff *skb); int skb_vlan_push(struct sk_buff *skb, __be16 vlan_proto, u16 vlan_tci); -static inline int memcpy_from_msg(void *data, struct msghdr *msg, int len) +static inline int memcpy_from_msg(void *data, struct msghdr *msg, size_t len) { return copy_from_iter(data, len, &msg->msg_iter) == len ? 0 : -EFAULT; } -static inline int memcpy_to_msg(struct msghdr *msg, void *data, int len) +static inline int memcpy_to_msg(struct msghdr *msg, void *data, size_t len) { return copy_to_iter(data, len, &msg->msg_iter) == len ? 0 : -EFAULT; } -- 2.4.10