From patchwork Sat Mar 12 20:28:32 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Sergey Shtylyov X-Patchwork-Id: 12778934 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 760B7C433EF for ; Sat, 12 Mar 2022 20:28:46 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232278AbiCLU3u (ORCPT ); Sat, 12 Mar 2022 15:29:50 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:42616 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229846AbiCLU3u (ORCPT ); Sat, 12 Mar 2022 15:29:50 -0500 Received: from mxout03.lancloud.ru (mxout03.lancloud.ru [45.84.86.113]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id B996B8EB44 for ; Sat, 12 Mar 2022 12:28:39 -0800 (PST) Received: from LanCloud DKIM-Filter: OpenDKIM Filter v2.11.0 mxout03.lancloud.ru 27461208F048 Received: from LanCloud Received: from LanCloud Received: from LanCloud From: Sergey Shtylyov To: Alan Stern , Greg Kroah-Hartman , Subject: [PATCH 0/2] Switch to using scnprintf() in the UHCI driver's debugging code Date: Sat, 12 Mar 2022 23:28:32 +0300 Message-ID: <20220312202834.11700-1-s.shtylyov@omp.ru> X-Mailer: git-send-email 2.26.3 MIME-Version: 1.0 X-Originating-IP: [192.168.11.198] X-ClientProxiedBy: LFEXT02.lancloud.ru (fd00:f066::142) To LFEX1907.lancloud.ru (fd00:f066::207) Precedence: bulk List-ID: X-Mailing-List: linux-usb@vger.kernel.org Here are 2 patches against the 'usb-next' branch of Greg KH's 'usb.git' repo. The UHCI driver's debugging code uses a lot of sprintf() calls with the large buffers, leaving some space at the end of the buffers to handle the buffer overflow. Using scnprntf() calls instead eliminates the very possibility of the buffer overflow... Sergey Shtylyov (2): usb: host: uhci-debug: use scnprintf() instead of sprintf() usb: host: uhci: remove #define EXTRA_SPACE drivers/usb/host/uhci-debug.c | 263 ++++++++++++---------------------- drivers/usb/host/uhci-hcd.c | 2 +- drivers/usb/host/uhci-q.c | 2 +- 3 files changed, 93 insertions(+), 174 deletions(-)