From patchwork Mon Jul 17 14:29:58 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Arnd Bergmann X-Patchwork-Id: 107967 Delivered-To: patch@linaro.org Received: by 10.140.101.44 with SMTP id t41csp4591666qge; Mon, 17 Jul 2017 07:30:43 -0700 (PDT) X-Received: by 10.84.132.40 with SMTP id 37mr16921946ple.149.1500301843161; Mon, 17 Jul 2017 07:30:43 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1500301843; cv=none; d=google.com; s=arc-20160816; b=pbf1Lly3XuGRUTLkXgd4D6OhTHUNl4FmB34mzSLBkwbm0FZAejqAGr2IoincW/bZ6v VJaouPgOS/PhVqraDcB1s4AuME6YKeC1KeHxMJmEEyL+WtfRs52vuwbBo49fhbqbIMrH LTorryHoAzfK8hiqHfJwdO9XTx6o8DAqUQ0KHlpakmMCX75nB/ptJFaBrAMj0FL7X+l1 I/vO4ZiCw+kcYd/Bkn3vCMO2d/ikO3EXipW6Za/SdIKtOgGr3vq8VIDEIRn2Wnh7D766 rsdivFy/RlHTO/jxoeCiDyc+h19AH6LZh3ZNSLWBOJGMraixmquirSBw/JCruUa92BO2 gSRg== 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=cUjR0dSK6QIwxDjiTBWoWqR/f/+AC8F9/tWtODnGBEo=; b=ueyW6QJ6RddmnZ3+sSR5vv5s9LrtYXK/8oLkl5A/P1+xuFRtAXcjroHCl5xm/++PYP 8Mun3FyMP6WggejjhiQl9VRlLqdkjW4I2QomDdRLtUyTfeWRu1/5rF8zDGC/Dfh43RkD S+wydrKPgJL0EG96KJAccwJDCJISOk31knYKkIdAr1iPZ/pzgsr0RlJY4Y9wC41uRPPo sK8X1W5fW0iSPPCcut7jmKXUFsWF2XKkoMc8n521qmAZfKiCQ+JtXm1k9JVX21R3eKBc 1D4C2y6d+yIpwV0QhzUtuZ7sJDEJ55RHtdSkIDq2cA0lCB39arcw34uyk5tiimOcoQo1 0/4Q== 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 t25si13171918pge.237.2017.07.17.07.30.42; Mon, 17 Jul 2017 07:30:43 -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 S1751530AbdGQOak (ORCPT + 25 others); Mon, 17 Jul 2017 10:30:40 -0400 Received: from mout.kundenserver.de ([212.227.126.130]:55229 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751278AbdGQOai (ORCPT ); Mon, 17 Jul 2017 10:30:38 -0400 Received: from wuerfel.lan ([5.56.224.194]) by mrelayeu.kundenserver.de (mreue004 [212.227.15.129]) with ESMTPA (Nemesis) id 0LfFcq-1dzcJX1QQC-00ojqZ; Mon, 17 Jul 2017 16:30:27 +0200 From: Arnd Bergmann To: Hans Verkuil , Mauro Carvalho Chehab Cc: Arnd Bergmann , Sakari Ailus , linux-media@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH] [v2] [media] usbvision-i2c: fix format overflow warning Date: Mon, 17 Jul 2017 16:29:58 +0200 Message-Id: <20170717143024.862161-1-arnd@arndb.de> X-Mailer: git-send-email 2.9.0 X-Provags-ID: V03:K0:5WKkg8wP20R/GTtomQDY+b7rb4DRcQLwvLb5A6s3n2lQbmULLDy pXAfLlGI27QpJ22t+atI71sUHIX0hXF4MA3PJPPTj8RNEZIOkGy6bsksac1pfRbPAogUGDG sg8BeS9ufzGniI7qFDDhSiteBDVm23VQYrzmvfFRVtYcFnuYxgEU+b5AZ7wYyAYBIj1/WEM tOXDtlBl5uzJtFKfDOFNA== X-UI-Out-Filterresults: notjunk:1; V01:K0:UH9kr4Idt2A=:tevAaSviuHlP0KHw60btEJ r09iaqGD1amu1cwjJKdrN66HmzbbEyTOhkfSncchlIN6XVYx8p6GwW2bmPFTXcB+pHpr6kRuf WSf3WVbZ9fPyanlaUyxpULsIhH7ikU8lSJtyCZDaI8pDCx4/q+D9Q+W0qVdDFZrYWBIk5U4AJ g1oRAW0CCVcdLqKGBRE2R91CagvMSzVkZZ3V2EjXsVqZdnCGhGk+9U8JXjkshii49ZgC/ykjb vtxwoKJCNWKu3eke1f3gbdq/pMAZh8OR4d3jJ3kfLpYkB+AmOhxoESF+uIeAKXKeDFXarMl1n Wr75cjiHXZAbQWKKX8lYl1KnKVZPGK93Pys9oMSOAh3dsXhiry1/dIWA0/8gzdoBjYC/qlDqK RuYU75uue30n4/qLIANKK7HNcnCCUGJuXHqOWbgBeY9a7aSFJnBuHvPl87SDdHihjF8eFdjvw qqjrzY0qMIO6C1R+p1DrUTE8JjGUITKavVK2+wvIdUuiqW9yJ/Bzyj7f4HrNhIETg2MZhT0fk soEcNEV10hZ/DOVG6OIrwPdBjLfkvuWNTelLtZ458UiGIkPme3H5flGtiNAN24oejDYm9Fv9b JUpcbB21v9OUW6naBmw/SR+aB+xK1JtKgmJ/N6htRJGgPRzMendhjwJ7CTQHVE9df3rpacw7V y8gG42NJGitgA1q7SyYKooX6iLmPfZZQCRMXSYrXWsaTY7SmeI3e9a/uOrppb3722vLlBePuf 9zqqLVYj5kJmkAuiOj5XiKYX/XpqYFPKZTFbOA== Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org gcc-7 notices that we copy a fixed length string into another string of the same size, with additional characters: drivers/media/usb/usbvision/usbvision-i2c.c: In function 'usbvision_i2c_register': drivers/media/usb/usbvision/usbvision-i2c.c:190:36: error: '%d' directive writing between 1 and 11 bytes into a region of size between 0 and 47 [-Werror=format-overflow=] sprintf(usbvision->i2c_adap.name, "%s-%d-%s", i2c_adap_template.name, ^~~~~~~~~~ drivers/media/usb/usbvision/usbvision-i2c.c:190:2: note: 'sprintf' output between 4 and 76 bytes into a destination of size 48 Using snprintf() makes the code more robust in general, but will still trigger a possible warning about truncation in the string. We know this won't happen as the template name is always "usbvision", so we can easily avoid the warning as well by using this as the format string directly. Signed-off-by: Arnd Bergmann --- v2: use snprintf() --- drivers/media/usb/usbvision/usbvision-i2c.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) -- 2.9.0 diff --git a/drivers/media/usb/usbvision/usbvision-i2c.c b/drivers/media/usb/usbvision/usbvision-i2c.c index fdf6b6e285da..38749331e7df 100644 --- a/drivers/media/usb/usbvision/usbvision-i2c.c +++ b/drivers/media/usb/usbvision/usbvision-i2c.c @@ -187,8 +187,9 @@ int usbvision_i2c_register(struct usb_usbvision *usbvision) usbvision->i2c_adap = i2c_adap_template; - sprintf(usbvision->i2c_adap.name, "%s-%d-%s", i2c_adap_template.name, - usbvision->dev->bus->busnum, usbvision->dev->devpath); + snprintf(usbvision->i2c_adap.name, sizeof(usbvision->i2c_adap.name), + "usbvision-%d-%s", + usbvision->dev->bus->busnum, usbvision->dev->devpath); PDEBUG(DBG_I2C, "Adaptername: %s", usbvision->i2c_adap.name); usbvision->i2c_adap.dev.parent = &usbvision->dev->dev;