From patchwork Fri Jul 6 13:28:32 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Arnd Bergmann X-Patchwork-Id: 141319 Delivered-To: patch@linaro.org Received: by 2002:a2e:9754:0:0:0:0:0 with SMTP id f20-v6csp3116775ljj; Fri, 6 Jul 2018 06:29:02 -0700 (PDT) X-Google-Smtp-Source: AAOMgpeFtUX6+Ej8DTSxBKON4oOdq60jkkE8cSXjnC10yeEPEOSfJBkeM2PD/fM0DXEnbZ8+Pez4 X-Received: by 2002:a17:902:102b:: with SMTP id b40-v6mr10286041pla.125.1530883742204; Fri, 06 Jul 2018 06:29:02 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1530883742; cv=none; d=google.com; s=arc-20160816; b=dCAZFP5sku8E/XpPzt93Pq0OJuKB7j1v/8gbX0eXr/q9eRtEXtvzQTuzb0O95j0QO8 mdafpHANMwawNWGiTi/tTsnfRe47oE3EKuGzrn75zlqwZOFGaPs1OunaHJEIQRdbN1q2 jANF744QptT9+PTTjGvXbxlsCUannsQ1k2JWm5XGH7xnbd9akZJTjnx04fzSKNPAZQwo p5VkNc6Lz64M43QI8QlCuzeArzuKaz+y3EBCEeiOv2BgrKLqyKv2c3ZXP9kviAmql8jo o92xqYiaHRmJ7UsNsdJvodHPWMMMgIiZeDx94VB69IiPAdBP6tR3dZv5YwbRPZOt7XE9 +D+g== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:message-id:date:subject:cc:to:from :arc-authentication-results; bh=66NDokyZbydQE1hmXrKo5mPzA0lGLcOZd88yyQDhDdg=; b=LbMH8CWx6UI3PqrL0M1QGsj5XqH2gsvaj9wsNmfeyhdBYYqIIjfYektY9xFPjKFXot QSsV7Csq1hO6y1+Rvj/jERDAw+ZjkkUw3PxX/yEqZ3KBVTwOa3ZgRlKhTKaiUiEL7Vdo ricjFogbQBGK9VsJKrHDrkZygDUQzGI26TNU6druIBUxvKYvvX2SNmaOjX3kfhLKFtwo /Y5HWtmSps9iurExRMD4vD8EpAzp5iqkM38Xev4xzh0/HfZEV/5196DtegKddPO4mR2d nSM+o+FnOs+UdwAOe/r837ZDK4JNrVZfVkrSXbkQ5b0AKp73Zkv16Tbr/2MWaO/aRx4T CWdA== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id 70-v6si2489210pfz.72.2018.07.06.06.29.01; Fri, 06 Jul 2018 06:29:02 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933684AbeGFN27 (ORCPT + 30 others); Fri, 6 Jul 2018 09:28:59 -0400 Received: from mout.kundenserver.de ([217.72.192.73]:34491 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933156AbeGFN2x (ORCPT ); Fri, 6 Jul 2018 09:28:53 -0400 Received: from wuerfel.lan ([46.223.138.35]) by mrelayeu.kundenserver.de (mreue103 [212.227.15.145]) with ESMTPA (Nemesis) id 0M5IbP-1gJm6p2EZz-00zaJY; Fri, 06 Jul 2018 15:28:48 +0200 From: Arnd Bergmann To: Heikki Krogerus , Greg Kroah-Hartman Cc: Arnd Bergmann , linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH] usb: typec: avoid format-overflow warning Date: Fri, 6 Jul 2018 15:28:32 +0200 Message-Id: <20180706132843.51206-1-arnd@arndb.de> X-Mailer: git-send-email 2.9.0 X-Provags-ID: V03:K1:67fAmhG4BXLgjXhZS+i3M+P5K0STs1gZMZ2cTV3wx0GFnSmMPGZ gSfb5Z225epl/DXVIAV+u83VDYHAhUcNnmCmV9E9Bu82YUi3SLbKNgDtV3aRBUlPHBvErCi ZUy4pLrhKC6XYzJXbnj1EJvMUHeFlZvt3cxQKNptSpKV9kj7Ow6cZx1vGOoBD9R1YNukPIO KnExyYawPqBcAi0Gn0RNg== X-UI-Out-Filterresults: notjunk:1; V01:K0:X5X8Ya6nl2w=:bahXS9MR84LA6ruL6kOmBF 65DRrqcxvapBWvvT3pIUCMLnWoZTxQrQR26pEx6Np+4HaqeGdaVrLIccYi1CmcqADG/p9KS/s DaX20GlbFNMAn3I9q/zFvg8UqiBOtpIm2L5+G2BCAsh0Wcawb3Qz7cBfzLh36RACPomXGm6Dg BOjVU5kvSLrHXU9ADTyczU9q8bk9DeQLZS9xxU6NYgTJu2Renm46RaRQylYWC15OVRd94+iVW F7z/wxGdwEBfOMivkzsRbmIKhDCgGClbjXaUE0CEsG7BxT7pjKqCSKdi//bHLKmd71frlOGKT toQgzsLpCl0BJRE74G4D64Vq3nH/7iD5/Gw8SDc0X+RjClhkjqRUH/vx2QuiLwZbt4J+z0Mgp vn0BKFzPlfOj+sdRV3GqJO8RHRTCvsjK6JeqPG4MtimFKWAeBVJRjeFWuRQ2sCn86G6TK18jQ YukkgJy/aWevkXy7/V/hKWwfWSktN3zHJ5cpK8x73TjM2iuCKvRDJYvAC+ui/ZkrWx7FU3LWu rQDMyy3ZAJQ8NeBcD1oaRErZwx0rHgULpdXHnHOncJDEZD5jQG1m0ko1qmcz3d3lo5wTKCcoD 6F5tHbAzyTREr0BRbFSnZedijtGjhJjGsV9eIg4TcsKL33StPxYJk+5dcKK9nJQP2DWCAQ466 eV+c4hH0lkLjYKa0awZ2GopA1TL8vYyGoomBH/qbqCL1+3oduL0T+i4eO27im2SuIqKs= Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org gcc-8 points out that the fix-byte buffer might be too small if desc->mode is a three-digit number: drivers/usb/typec/class.c: In function 'typec_register_altmode': drivers/usb/typec/class.c:502:32: error: '%d' directive writing between 1 and 3 bytes into a region of size 2 [-Werror=format-overflow=] sprintf(alt->group_name, "mode%d", desc->mode); ^~ drivers/usb/typec/class.c:502:27: note: directive argument in the range [0, 255] sprintf(alt->group_name, "mode%d", desc->mode); ^~~~~~~~ drivers/usb/typec/class.c:502:2: note: 'sprintf' output between 6 and 8 bytes into a destination of size 6 sprintf(alt->group_name, "mode%d", desc->mode); ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ I assume this cannot happen in practice, but we can simply make the string long enough to avoid the warning. This uses the two padding bytes that already exist after the string. Fixes: 4ab8c18d4d67 ("usb: typec: Register a device for every mode") Signed-off-by: Arnd Bergmann --- drivers/usb/typec/bus.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) -- 2.9.0 Acked-by: Heikki Krogerus diff --git a/drivers/usb/typec/bus.h b/drivers/usb/typec/bus.h index 62aaf8b56bde..db40e61d8b72 100644 --- a/drivers/usb/typec/bus.h +++ b/drivers/usb/typec/bus.h @@ -16,7 +16,7 @@ struct altmode { enum typec_port_data roles; struct attribute *attrs[5]; - char group_name[6]; + char group_name[8]; struct attribute_group group; const struct attribute_group *groups[2];