Message ID | MWHPR17MB1662D37AF5ADBD0789E5C05CA07E0@MWHPR17MB1662.namprd17.prod.outlook.com |
---|---|
State | Accepted |
Headers | show
Return-Path: <lede-dev-bounces+incoming=patchwork.ozlabs.org@lists.infradead.org> X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from bombadil.infradead.org (unknown [65.50.211.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ozlabs.org (Postfix) with ESMTPS id 3v43cl6FPPz9ssP for <incoming@patchwork.ozlabs.org>; Thu, 19 Jan 2017 23:51:35 +1100 (AEDT) Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux)) id 1cUCBg-0000VT-Gq; Thu, 19 Jan 2017 12:51:32 +0000 Received: from col004-omc2s6.hotmail.com ([65.55.34.80]) by bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux)) id 1cUCBc-0000Pq-75 for lede-dev@lists.infradead.org; Thu, 19 Jan 2017 12:51:30 +0000 Received: from NAM04-SN1-obe.outbound.protection.outlook.com ([65.55.34.73]) by COL004-OMC2S6.hotmail.com over TLS secured channel with Microsoft SMTPSVC(7.5.7601.23008); Thu, 19 Jan 2017 04:51:05 -0800 Received: from CO1NAM04FT049.eop-NAM04.prod.protection.outlook.com (10.152.90.52) by CO1NAM04HT082.eop-NAM04.prod.protection.outlook.com (10.152.91.148) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P384) id 15.1.803.8; Thu, 19 Jan 2017 12:51:04 +0000 Received: from MWHPR17MB1662.namprd17.prod.outlook.com (10.152.90.58) by CO1NAM04FT049.mail.protection.outlook.com (10.152.91.187) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P384) id 15.1.803.8 via Frontend Transport; Thu, 19 Jan 2017 12:51:04 +0000 Received: from MWHPR17MB1662.namprd17.prod.outlook.com ([10.173.242.140]) by MWHPR17MB1662.namprd17.prod.outlook.com ([10.173.242.140]) with mapi id 15.01.0860.012; Thu, 19 Jan 2017 12:51:04 +0000 From: Mihai Richard <mihairichard@live.com> To: "lede-dev@lists.infradead.org" <lede-dev@lists.infradead.org> Thread-Topic: [PATCH] ubusd: fix issue caused by an implicit cast Thread-Index: AQHSclKx4FfYSzWvu0mpRWU0/8YceQ== Date: Thu, 19 Jan 2017 12:51:04 +0000 Message-ID: <MWHPR17MB1662D37AF5ADBD0789E5C05CA07E0@MWHPR17MB1662.namprd17.prod.outlook.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-slblob-mailprops: BBDuhOFk7KBCWbGHTt/AD/pKTWxStbog1WKU/m07JGAnm8cnfIftKXYFlNnIuwClz5SUqppIRNP93mtDbIPPniMwe0LAwPqwBiaAYcl1OarqaM3kVtPdB4fOuVVMhKFnXFDaTZs7WyNaBsrcFVlGl8bTBc+dhl5POFkoBp9u9LvLLcs42DLBo/w8CmLsmckbUgatZhDR62BrgQmf9a0TFVx3mDjVxOHIdKbBCQiwFQSZXP8rxYHiNMq67ZrRcbiKCSCvrCWUG2O7LkEy1MGYOpw9DMPT6ZcWswEPUhgtKq9HKtaQmFpoejT6QQ9dsB3h78Nr0haWAFqQoilZXZV/ghXfRoO1hZJabRxIO3fg0/HG3R7Ao3XoQvMKi+vGmfI+1MKQHZsX6B9BnmiV55DivihHwaz3dXiN/fdpPS2RloTV9aIu/Wqj8e9PiCKhndfhkSHcGgjC+FvyCTYn7q/vnbapAJuAc08cOYk0AgQ6Vv0pVyKviJmCMyZhRxBkpOv2lN6CAyiit9u7wP1RhAj1lYvVrI8VmFjIKdqf1WUHIjvd0HjwQrgV2Q2IsSYfVw22RuuG6njfrK2bSiQ5OZG6ub4yI3t8d1SAtSv7iVZEyE1ihiwihf3cevYiSBRPei03PQXQbmUiMQhC5vwzeyu6bb+TEMYupSg+Ivm70ohVbgk= authentication-results: lists.infradead.org; dkim=none (message not signed) header.d=none; lists.infradead.org; dmarc=none action=none header.from=live.com; x-incomingtopheadermarker: OriginalChecksum:3FDF043C6B52425F245A63A2D55519751662AA77F8A9F7C8D8DEAB58E29739A8; UpperCasedChecksum:63FA493EF035666464F7A169C2104893D3CCE9EA3AC2F2DFEC251C6DE65054D1; SizeAsReceived:8104; Count:36 x-ms-exchange-messagesentrepresentingtype: 1 x-incomingheadercount: 36 x-eopattributedmessage: 0 x-microsoft-exchange-diagnostics: 1; CO1NAM04HT082; 7:jK7wsZaVK6+GIYXLld+gBWvV993jeVcZcLgP6MU5qgRk67fWwh4FVAILc0o13L4e9ftBiaZj8ztI7kH5Vci6i6DNNxUGidUzheB4t2566JiQEEHYgfF8ybaxYYA16RyNbeCEfP7084HrzSTBq4bEESGSkjIJeSv4NY1pVYhYCb2QZRBR77XiiveJMk2S3JitAQFOnjlkohd8/xyv/E6TnGW0zmrb+d8pmTI9Kgg1O9F8iFH5NzP0imnyGc/V0mG1OOLfqSz5Gy+GV4Y4T3E/HkdhD4LJJ+gQeHOLIj5CrXAJE8wMms/OJE6PjGzPnyhwed9+KV/cOxvI0j8X2cTQzzfhA/+hTZXCB09457mWBhvmFHxAnAloU67CNoe5s2ST7EBS9P9KB2mYk55deaLk35ySnphGVLvJS/IWQGKSV+Vp0pzDpgnl2QusiyPHeMVK08oFvMIirWtmDUiCk2bc1g== x-forefront-antispam-report: EFV:NLI; SFV:NSPM; SFS:(10019020)(98900005); DIR:OUT; SFP:1102; SCL:1; SRVR:CO1NAM04HT082; H:MWHPR17MB1662.namprd17.prod.outlook.com; FPR:; SPF:None; LANG:en; x-ms-office365-filtering-correlation-id: 3d1cec52-ad3f-4f9a-ad6d-08d44069d41e x-microsoft-antispam: UriScan:; BCL:0; PCL:0; RULEID:(22001)(1601124038)(5061506344)(5061507293)(1603103113)(1603101340)(1601125047)(1701031023); SRVR:CO1NAM04HT082; x-exchange-antispam-report-cfa-test: BCL:0; PCL:0; RULEID:(444111334)(444112120)(432015012)(82015046); SRVR:CO1NAM04HT082; BCL:0; PCL:0; RULEID:; SRVR:CO1NAM04HT082; x-forefront-prvs: 0192E812EC spamdiagnosticoutput: 1:99 spamdiagnosticmetadata: NSPM MIME-Version: 1.0 X-OriginatorOrg: live.com X-MS-Exchange-CrossTenant-originalarrivaltime: 19 Jan 2017 12:51:04.4049 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Internet X-MS-Exchange-CrossTenant-id: 84df9e7f-e9f6-40af-b435-aaaaaaaaaaaa X-MS-Exchange-Transport-CrossTenantHeadersStamped: CO1NAM04HT082 X-OriginalArrivalTime: 19 Jan 2017 12:51:05.0880 (UTC) FILETIME=[B2BD4980:01D27252] X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20170119_045128_314056_8218BFA8 X-CRM114-Status: UNSURE ( 7.06 ) X-CRM114-Notice: Please train this message. X-Spam-Score: -3.8 (---) X-Spam-Report: SpamAssassin version 3.4.1 on bombadil.infradead.org summary: Content analysis details: (-3.8 points) pts rule name description ---- ---------------------- -------------------------------------------------- -0.7 RCVD_IN_DNSWL_LOW RBL: Sender listed at http://www.dnswl.org/, low trust [65.55.34.80 listed in list.dnswl.org] 0.0 FREEMAIL_FROM Sender email is commonly abused enduser mail provider (mihairichard[at]live.com) -0.0 SPF_PASS SPF: sender matches SPF record -1.2 RCVD_IN_MSPIKE_H2 RBL: Average reputation (+2) [65.55.34.80 listed in wl.mailspike.net] -1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1% [score: 0.0000] Subject: [LEDE-DEV] [PATCH] ubusd: fix issue caused by an implicit cast X-BeenThere: lede-dev@lists.infradead.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: <lede-dev.lists.infradead.org> List-Unsubscribe: <http://lists.infradead.org/mailman/options/lede-dev>, <mailto:lede-dev-request@lists.infradead.org?subject=unsubscribe> List-Archive: <http://lists.infradead.org/pipermail/lede-dev/> List-Post: <mailto:lede-dev@lists.infradead.org> List-Help: <mailto:lede-dev-request@lists.infradead.org?subject=help> List-Subscribe: <http://lists.infradead.org/mailman/listinfo/lede-dev>, <mailto:lede-dev-request@lists.infradead.org?subject=subscribe> Cc: Mihai Richard <mihairichard@live.com> Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "Lede-dev" <lede-dev-bounces@lists.infradead.org> Errors-To: lede-dev-bounces+incoming=patchwork.ozlabs.org@lists.infradead.org |
diff --git a/ubusd.c b/ubusd.c index 7279a70..5409b7f 100644 --- a/ubusd.c +++ b/ubusd.c @@ -148,12 +148,13 @@ void ubus_msg_send(struct ubus_client *cl, struct ubus_msg_buf *ub, bool free) if (!cl->tx_queue[cl->txq_cur]) { written = ubus_msg_writev(cl->sock.fd, ub, 0); - if (written >= ub->len + sizeof(ub->hdr)) - goto out; if (written < 0) written = 0; + if (written >= ub->len + sizeof(ub->hdr)) + goto out; + cl->txq_ofs = written; /* get an event once we can write to the socket again */
An -1 returned by ubus_msg_writev() will be interpreted as UINT_MAX during a check to see how much data had could be written on the socket. Because sizeof() will return size_t it will promote the comparsion to unsigned Signed-off-by: Mihai Richard <mihairichard@live.com> --- ubusd.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-)