{"id":807958,"url":"http://patchwork.ozlabs.org/api/1.2/patches/807958/?format=json","web_url":"http://patchwork.ozlabs.org/project/uboot/patch/1504132946-30890-1-git-send-email-joe.hershberger@ni.com/","project":{"id":18,"url":"http://patchwork.ozlabs.org/api/1.2/projects/18/?format=json","name":"U-Boot","link_name":"uboot","list_id":"u-boot.lists.denx.de","list_email":"u-boot@lists.denx.de","web_url":null,"scm_url":null,"webscm_url":null,"list_archive_url":"","list_archive_url_format":"","commit_url_format":""},"msgid":"<1504132946-30890-1-git-send-email-joe.hershberger@ni.com>","list_archive_url":null,"date":"2017-08-30T22:42:26","name":"[U-Boot] net: Correct size of NFS buffers","commit_ref":null,"pull_url":null,"state":"changes-requested","archived":false,"hash":"723d5384f67a29d983400c70226db6ed044141c9","submitter":{"id":8809,"url":"http://patchwork.ozlabs.org/api/1.2/people/8809/?format=json","name":"Joe Hershberger","email":"joe.hershberger@ni.com"},"delegate":{"id":6337,"url":"http://patchwork.ozlabs.org/api/1.2/users/6337/?format=json","username":"jhersh","first_name":"Joe","last_name":"Hershberger","email":"joe.hershberger@gmail.com"},"mbox":"http://patchwork.ozlabs.org/project/uboot/patch/1504132946-30890-1-git-send-email-joe.hershberger@ni.com/mbox/","series":[{"id":716,"url":"http://patchwork.ozlabs.org/api/1.2/series/716/?format=json","web_url":"http://patchwork.ozlabs.org/project/uboot/list/?series=716","date":"2017-08-30T22:42:26","name":"[U-Boot] net: Correct size of NFS buffers","version":1,"mbox":"http://patchwork.ozlabs.org/series/716/mbox/"}],"comments":"http://patchwork.ozlabs.org/api/patches/807958/comments/","check":"pending","checks":"http://patchwork.ozlabs.org/api/patches/807958/checks/","tags":{},"related":[],"headers":{"Return-Path":"<u-boot-bounces@lists.denx.de>","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":"ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.denx.de\n\t(client-ip=81.169.180.215; helo=lists.denx.de;\n\tenvelope-from=u-boot-bounces@lists.denx.de;\n\treceiver=<UNKNOWN>)","Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3xjLDG0svWz9s83\n\tfor <incoming@patchwork.ozlabs.org>;\n\tThu, 31 Aug 2017 08:44:46 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid CF810C21CB6; Wed, 30 Aug 2017 22:44:37 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id 27D34C21C39;\n\tWed, 30 Aug 2017 22:44:35 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid 0A507C21C39; Wed, 30 Aug 2017 22:44:33 +0000 (UTC)","from mx0b-00010702.pphosted.com (mx0a-00010702.pphosted.com\n\t[148.163.156.75])\n\tby lists.denx.de (Postfix) with ESMTPS id 633BCC21C26\n\tfor <u-boot@lists.denx.de>; Wed, 30 Aug 2017 22:44:33 +0000 (UTC)","from pps.filterd (m0098780.ppops.net [127.0.0.1])\n\tby mx0a-00010702.pphosted.com (8.16.0.21/8.16.0.21) with SMTP id\n\tv7UMf0AU008190; Wed, 30 Aug 2017 17:44:31 -0500","from ni.com (skprod3.natinst.com [130.164.80.24])\n\tby mx0a-00010702.pphosted.com with ESMTP id 2cny6h2p6x-1\n\t(version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256\n\tverify=NOT); Wed, 30 Aug 2017 17:44:31 -0500","from us-aus-exch2.ni.corp.natinst.com\n\t(us-aus-exch2.ni.corp.natinst.com [130.164.68.12])\n\tby us-aus-skprod3.natinst.com (8.16.0.21/8.16.0.21) with ESMTPS id\n\tv7UMiUcv027568\n\t(version=TLSv1.2 cipher=ECDHE-RSA-AES256-SHA384 bits=256 verify=NOT); \n\tWed, 30 Aug 2017 17:44:30 -0500","from us-aus-exch4.ni.corp.natinst.com (130.164.68.14) by\n\tus-aus-exch2.ni.corp.natinst.com (130.164.68.12) with Microsoft SMTP\n\tServer (TLS) id 15.0.1156.6; Wed, 30 Aug 2017 17:44:30 -0500","from us-aus-exhub2.ni.corp.natinst.com (130.164.68.32) by\n\tus-aus-exch4.ni.corp.natinst.com (130.164.68.14) with Microsoft SMTP\n\tServer (TLS) id 15.0.1156.6; Wed, 30 Aug 2017 17:44:30 -0500","from linux-xvxi.natinst.com (130.164.49.7) by\n\tus-aus-exhub2.ni.corp.natinst.com (130.164.68.32) with Microsoft SMTP\n\tServer\n\tid 15.0.1156.6 via Frontend Transport; Wed, 30 Aug 2017 17:44:30 -0500"],"X-Spam-Checker-Version":"SpamAssassin 3.4.0 (2014-02-07) on lists.denx.de","X-Spam-Level":"","X-Spam-Status":"No, score=-0.7 required=5.0 tests=RCVD_IN_DNSWL_LOW\n\tautolearn=unavailable autolearn_force=no version=3.4.0","From":"Joe Hershberger <joe.hershberger@ni.com>","To":"<u-boot@lists.denx.de>","Date":"Wed, 30 Aug 2017 17:42:26 -0500","Message-ID":"<1504132946-30890-1-git-send-email-joe.hershberger@ni.com>","X-Mailer":"git-send-email 1.7.11.5","MIME-Version":"1.0","X-Proofpoint-Virus-Version":"vendor=fsecure engine=2.50.10432:, ,\n\tdefinitions=2017-08-30_09:, , signatures=0","X-Proofpoint-Spam-Details":"rule=outbound_policy_notspam\n\tpolicy=outbound_policy score=30\n\tpriorityscore=1501 malwarescore=0 suspectscore=4 phishscore=0\n\tbulkscore=0\n\tspamscore=0 clxscore=1015 lowpriorityscore=0 impostorscore=0\n\tadultscore=0\n\tclassifier=spam adjust=30 reason=mlx scancount=1\n\tengine=8.0.1-1707230000\n\tdefinitions=main-1708300345","Cc":"Tom Rini <trini@konsulko.com>, Joe Hershberger <joe.hershberger@ni.com>","Subject":"[U-Boot] [PATCH] net: Correct size of NFS buffers","X-BeenThere":"u-boot@lists.denx.de","X-Mailman-Version":"2.1.18","Precedence":"list","List-Id":"U-Boot discussion <u-boot.lists.denx.de>","List-Unsubscribe":"<https://lists.denx.de/options/u-boot>,\n\t<mailto:u-boot-request@lists.denx.de?subject=unsubscribe>","List-Archive":"<http://lists.denx.de/pipermail/u-boot/>","List-Post":"<mailto:u-boot@lists.denx.de>","List-Help":"<mailto:u-boot-request@lists.denx.de?subject=help>","List-Subscribe":"<https://lists.denx.de/listinfo/u-boot>,\n\t<mailto:u-boot-request@lists.denx.de?subject=subscribe>","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" <u-boot-bounces@lists.denx.de>"},"content":"Reported-by: Coverity (CID: 152888)\nSigned-off-by: Joe Hershberger <joe.hershberger@ni.com>\n---\n\n net/nfs.h | 4 ++--\n 1 file changed, 2 insertions(+), 2 deletions(-)","diff":"diff --git a/net/nfs.h b/net/nfs.h\nindex 70a1a6d..a773cfb 100644\n--- a/net/nfs.h\n+++ b/net/nfs.h\n@@ -59,7 +59,7 @@ enum rpc_accept_stat {\n \n struct rpc_t {\n \tunion {\n-\t\tuint8_t data[2048];\n+\t\tuint8_t data[NFS_READ_SIZE + 6 * sizeof(uint32_t)];\n \t\tstruct {\n \t\t\tuint32_t id;\n \t\t\tuint32_t type;\n@@ -76,7 +76,7 @@ struct rpc_t {\n \t\t\tuint32_t verifier;\n \t\t\tuint32_t v2;\n \t\t\tuint32_t astatus;\n-\t\t\tuint32_t data[NFS_READ_SIZE];\n+\t\t\tuint32_t data[NFS_READ_SIZE / sizeof(uint32_t)];\n \t\t} reply;\n \t} u;\n } __attribute__((packed));\n","prefixes":["U-Boot"]}