From patchwork Thu Feb 10 02:14:22 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Masahiro Yamada X-Patchwork-Id: 12741184 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 2072CC433EF for ; Thu, 10 Feb 2022 02:17:53 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231316AbiBJCRt (ORCPT ); Wed, 9 Feb 2022 21:17:49 -0500 Received: from mxb-00190b01.gslb.pphosted.com ([23.128.96.19]:38326 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230372AbiBJCRs (ORCPT ); Wed, 9 Feb 2022 21:17:48 -0500 Received: from condef-10.nifty.com (condef-10.nifty.com [202.248.20.75]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 258F222B14 for ; Wed, 9 Feb 2022 18:17:49 -0800 (PST) Received: from conuserg-12.nifty.com ([10.126.8.75])by condef-10.nifty.com with ESMTP id 21A2Ex5s020843 for ; Thu, 10 Feb 2022 11:14:59 +0900 Received: from localhost.localdomain (133-32-232-101.west.xps.vectant.ne.jp [133.32.232.101]) (authenticated) by conuserg-12.nifty.com with ESMTP id 21A2ERFX000422; Thu, 10 Feb 2022 11:14:27 +0900 DKIM-Filter: OpenDKIM Filter v2.10.3 conuserg-12.nifty.com 21A2ERFX000422 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nifty.com; s=dec2015msa; t=1644459268; bh=piOL58+rcWqAFtD7Ov4m4Iy5XUSFEbl9fS0pyq8+xvg=; h=From:To:Cc:Subject:Date:From; b=IYyqGdfMklNoigh29IPDO82qWHnoTzdw64ecvcqRW/SZxsMWYCWCePkuP4OGW9ASw txQWNmI7bnstgaDixe37CzQxwODb4zBkenGcckLq4XJi8w4SWadlC1TE7/IRXJRs7Q vvLziDcovZaV2Fb5uMm6JrsfmK0XYmmQ0+mg50aF/djZyEC7slRTaMp8ohgDD6V1pG qa+L+45x1Ff5LY4CQo0exFQHhZB3dGSVdPPkuFyrVv/nQcO2cQPxJlHAHQnmMhiIr5 LDCHr+7DWDfJL+BEsSyG5RusBcZ010lEDj7EDJ4/1U32ezBb/EdKePuDeAYaTrbAvH bRLlev6Co2KCg== X-Nifty-SrcIP: [133.32.232.101] From: Masahiro Yamada To: linux-modules@vger.kernel.org Cc: Masahiro Yamada Subject: [PATCH 1/2] modprobe: fix the NULL-termination of new_argv Date: Thu, 10 Feb 2022 11:14:22 +0900 Message-Id: <20220210021423.3388827-1-masahiroy@kernel.org> X-Mailer: git-send-email 2.32.0 MIME-Version: 1.0 Precedence: bulk List-ID: The number of new arguments is (i + argc - 1) as it is set to *p_argc one line below. The correct location of NULL termination is new_argv[i + argc - 1]. Signed-off-by: Masahiro Yamada --- tools/modprobe.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/modprobe.c b/tools/modprobe.c index 9387537..b6b2947 100644 --- a/tools/modprobe.c +++ b/tools/modprobe.c @@ -743,7 +743,7 @@ static char **prepend_options_from_env(int *p_argc, char **orig_argv) } memcpy(new_argv + i, orig_argv + 1, sizeof(char *) * (argc - 1)); - new_argv[i + argc] = NULL; + new_argv[i + argc - 1] = NULL; *p_argc = i + argc - 1; return new_argv;