From nobody Fri Mar 29 08:33:21 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) client-ip=192.237.175.120; envelope-from=xen-devel-bounces@lists.xenproject.org; helo=lists.xenproject.org; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) smtp.mailfrom=xen-devel-bounces@lists.xenproject.org; arc=pass (i=1 dmarc=pass fromdomain=suse.com); dmarc=pass(p=quarantine dis=none) header.from=suse.com ARC-Seal: i=2; a=rsa-sha256; t=1637317306; cv=pass; d=zohomail.com; s=zohoarc; b=dEwtbAmBKGKByNS1dSf0FWNq9A7SAtkuYoH0t7+vXaDWdYm5Nk5sJHWsixj/NIiD+jQGbMy+IpppSgLBJDK9q3X+zXzVSkBmIpu03PtaNSF3+6dSJMcQXB5JJ363KON9Z8Ni7rIDiJdTpnje3XsaJXjFBm97wbaJ/YK2XnBBN7c= ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1637317306; h=Content-Type:Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:To; bh=Z0rK9VmEi1jVsksstUzQ7TXl9Hlmh2BpCcX4s9+HTHg=; b=GEOms1H0/L08a3VsDogoWZf9VtBfieenwHuy2H+QodaPNSZa6goLvg4uq1eWlhv2sallbxAfpoS95lWzH1V9nLXEdekqgN7paOK3qvu/nMaNExss2EWk348iEidACXMP0cCt1lYeELfXYlrDRTsGZJnP1dcxTCZHqRkdRIcbT0o= ARC-Authentication-Results: i=2; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) smtp.mailfrom=xen-devel-bounces@lists.xenproject.org; arc=pass (i=1 dmarc=pass fromdomain=suse.com); dmarc=pass header.from= (p=quarantine dis=none) Return-Path: Received: from lists.xenproject.org (lists.xenproject.org [192.237.175.120]) by mx.zohomail.com with SMTPS id 1637317306769945.7716901952333; Fri, 19 Nov 2021 02:21:46 -0800 (PST) Received: from list by lists.xenproject.org with outflank-mailman.227720.393958 (Exim 4.92) (envelope-from ) id 1mo115-0001tR-Oi; Fri, 19 Nov 2021 10:21:11 +0000 Received: by outflank-mailman (output) from mailman id 227720.393958; Fri, 19 Nov 2021 10:21:11 +0000 Received: from localhost ([127.0.0.1] helo=lists.xenproject.org) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1mo115-0001tK-Li; Fri, 19 Nov 2021 10:21:11 +0000 Received: by outflank-mailman (input) for mailman id 227720; Fri, 19 Nov 2021 10:21:10 +0000 Received: from se1-gles-flk1-in.inumbo.com ([94.247.172.50] helo=se1-gles-flk1.inumbo.com) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1mo114-0001si-7T for xen-devel@lists.xenproject.org; Fri, 19 Nov 2021 10:21:10 +0000 Received: from de-smtp-delivery-102.mimecast.com (de-smtp-delivery-102.mimecast.com [194.104.111.102]) by se1-gles-flk1.inumbo.com (Halon) with ESMTPS id 69b7166a-4922-11ec-a9d2-d9f7a1cc8784; Fri, 19 Nov 2021 11:21:09 +0100 (CET) Received: from EUR02-AM5-obe.outbound.protection.outlook.com (mail-am5eur02lp2053.outbound.protection.outlook.com [104.47.4.53]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id de-mta-16-PM4tJkMSMSuCZqGyvd_4Ng-1; Fri, 19 Nov 2021 11:21:08 +0100 Received: from VI1PR04MB5600.eurprd04.prod.outlook.com (2603:10a6:803:e7::16) by VI1PR0402MB2704.eurprd04.prod.outlook.com (2603:10a6:800:b5::13) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4713.22; Fri, 19 Nov 2021 10:21:07 +0000 Received: from VI1PR04MB5600.eurprd04.prod.outlook.com ([fe80::8062:d7cb:ca45:1898]) by VI1PR04MB5600.eurprd04.prod.outlook.com ([fe80::8062:d7cb:ca45:1898%3]) with mapi id 15.20.4713.022; Fri, 19 Nov 2021 10:21:07 +0000 Received: from [10.156.60.236] (37.24.206.209) by AM6P191CA0067.EURP191.PROD.OUTLOOK.COM (2603:10a6:209:7f::44) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4713.20 via Frontend Transport; Fri, 19 Nov 2021 10:21:06 +0000 X-Outflank-Mailman: Message body and most headers restored to incoming version X-BeenThere: xen-devel@lists.xenproject.org List-Id: Xen developer discussion List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Errors-To: xen-devel-bounces@lists.xenproject.org Precedence: list Sender: "Xen-devel" X-Inumbo-ID: 69b7166a-4922-11ec-a9d2-d9f7a1cc8784 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.com; s=mimecast20200619; t=1637317269; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=Z0rK9VmEi1jVsksstUzQ7TXl9Hlmh2BpCcX4s9+HTHg=; b=GkIZNOP6qlBJWvuvZha9gtShbhXb6y48SJ/72ezzupA9rc6UUrXnlbm/NbxAGaGntsWhWt e8AjajJZzSxB7PMd7cS97+5j3XgXppkJwqrWt+1EPYEmfnD4dUbYtqnuXHp50mwV9yZHfW bcp9w1qU6GYFOTlHQMQ8SkJttf2ypr0= X-MC-Unique: PM4tJkMSMSuCZqGyvd_4Ng-1 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=ZdDwwF7yPgtmUB2LY1Mq9lo8oMgv4gyOly+Q2n5U/4+8gCvqbjB8RkgfNoVumNnw9gv6Xxv3otJCoZxiyHZis71ZPFSHx6NyBVG5BST/2CxyRL/u73XQ73LGxCQT2bNJE4TO38AzmhkP0B9f+3x/CVEBtgFc5nNZvuvvaOswjAbm02LJJWfWRdLxV4i2ziiHbAv60GtmyorUtuu62DbXEwUhHaj7BM6l+u+tQEE02snw1RYRNiFVrRY9LRUbPzt4bhFQsllD24i2wEWS0K0GiACJVILgrquSu+B026/LOvI5TVUtj1INNSDf8vERxOI6JD77YStOnIsLMqklYwZTmg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=Z0rK9VmEi1jVsksstUzQ7TXl9Hlmh2BpCcX4s9+HTHg=; b=iE0rm04yOlcGyFmDfn0Oyde/fslqXg9HUMESizOLSzph+Omr0cU36uaTB70/8OmuUTgGMBi0nHbfrmbebPAd5X8JwbjrclH2Sm+bUqXByAilHj8SRi3936rxXuBlJ0XYZmhh6HRyWp0AJ3sqHVjw7TCiqpdh/kqs3nhwMAOcLgcEALB/1uahFFEH04F9TQSHCzJA0DTkOEv3WLvidFwSwJmpitTxrklSFK80PG7nY8FDWzu9zONxWxmT3Qu97m+60T8CIZVDVi+yTQmdRQoOyBpJGa5q1IARH0/YmFBVjMs/4FqGJX6R8VgCudV29u2uawqL7n/FLIJq5zSM3VXP/g== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=suse.com; dmarc=pass action=none header.from=suse.com; dkim=pass header.d=suse.com; arc=none Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=suse.com; Message-ID: <0ed245fa-58a7-a5f6-b82e-48f9ed0b6970@suse.com> Date: Fri, 19 Nov 2021 11:21:06 +0100 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:91.0) Gecko/20100101 Thunderbird/91.3.0 Subject: [PATCH 1/7] xz: add fall-through comments to a switch statement Content-Language: en-US From: Jan Beulich To: "xen-devel@lists.xenproject.org" Cc: Andrew Cooper , George Dunlap , Ian Jackson , Julien Grall , Stefano Stabellini , Wei Liu References: In-Reply-To: Content-Transfer-Encoding: quoted-printable X-ClientProxiedBy: AM6P191CA0067.EURP191.PROD.OUTLOOK.COM (2603:10a6:209:7f::44) To VI1PR04MB5600.eurprd04.prod.outlook.com (2603:10a6:803:e7::16) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: bf198489-fb39-4abd-d27c-08d9ab464c9b X-MS-TrafficTypeDiagnostic: VI1PR0402MB2704: X-Microsoft-Antispam-PRVS: X-MS-Oob-TLC-OOBClassifiers: OLM:196; X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: 2K0QjACt30PfqxoI6Kx1MCGJBaXNCUMfxFQwYDNxBEeoTfUWSMs1Y7hh3b304tocW3cJ15jiUFWgMqJ09qdAuoY0rd7FpANE8SF/ptM9Z4OnsfRI09npDf0qdg55JV2KUGF6i8fRERbUd6pT/rqjRbi6pXMh8Wt7GoSDgoNYc1usVInubQjKYYmrEof6DB/xy4nyZG30bpSxhi4iZW6eW6a8Lsz4lN36RdwEQkRX8pxb1baE9a9DQtUFFNMdBo/DmV5sFTgAi8qxP2uGBQOy9a+M2Bp0XjsZDOaonlsl4tKjn8UjudKO7P30GBmYKY0WM9EQwYvSiG5lqnX8YcbB1tYWKARenKC14p/CwnoQDgug3jektZz7l0cF3wwTk/Aj3fTRoLaCHvEC9iIR4nmEJD3uD4CubvBcXH+1gInBmc+5FpD20BDZvVWBOn9QwLeBTKl7znf5+8PmFGngwYh9UDDf+b27JaccbwxSeKFYGSyp1QnNPuJDmQDh3nE5yjoboNu2ezbqdT9NFX1TNh9vG2pm8IeJ9hEPxurpHNs35h7dKWmdkmCvGRcJKYm2Ie2BekUhifC+aBnhnpI1tqcBsy/vTNSwBSFnEWV+vhIFiTFHlGNw037za75FLfsb9QogqvQINkJl0VlLnf4oggAcB0PSEQNVDodw1gwhEumrf2cOE8FtNBT16ifYVS1uiPuE4TyhP6TzdDB93fbJirns8QAMJNEM0QkAFIUxI7Qb5bcVIin17lZ4T2x17C7GEJKU X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:VI1PR04MB5600.eurprd04.prod.outlook.com;PTR:;CAT:NONE;SFS:(366004)(31696002)(38100700002)(36756003)(508600001)(2906002)(31686004)(8676002)(83380400001)(66476007)(66556008)(86362001)(956004)(2616005)(5660300002)(26005)(8936002)(6486002)(186003)(316002)(66946007)(54906003)(6916009)(16576012)(4326008)(45980500001)(43740500002);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?ZTAzd3lRQ01xWGtLQ1JDeWdXN21HU1F4UTNUN3R6emNOYUU2dXdNK0xMd3ph?= =?utf-8?B?clJaNEZTWE04SnAraldOUWp1VEZpNEY4WFExQlNYaVBoN25mNitTQlNFQWFa?= =?utf-8?B?UmU4ZGhyVm0wVHlGeUhMQ1hhYWRLcmUycmRHeGdWUzhKWjdCTmxWekNNRE8x?= =?utf-8?B?elJycGtqWWRLcEQrNTFNOEVHbHV2bWI5RThsVUsreFZwRUxBSXZ5emJXWXd0?= =?utf-8?B?QjM3SU1adWpxWTlWVlJpY3k4MzZ4VWVFUlZFVURMTGVqQ1U4TEZVU1RFbEx1?= =?utf-8?B?RmdoTkJnYllVQkJjbXFYaWdNWVJJbnh5VUxuaTVHNDRRdTlydXYxVExQdWx1?= =?utf-8?B?WUVCTHpCRWJtc01UQlgvRDZZSmpodjZVaUhYTUVCNkhNYXphZVNmS2tubXlJ?= =?utf-8?B?NDZ1eDJ3WU5LalAxdlBRQVk3SjJGZnFRNmxzaWZPczcwa0E3ZzdVcEVESHY2?= =?utf-8?B?dGFkK2lYUmdQNVBKTWdhd2JNcDFYU3FXK214UHpsUzRvUnVOd0g3bThKRjB5?= =?utf-8?B?cUhiaklwNnk2bGJyNEZncmFUbWIwYTNPNHNWdk5nbUdmeENqYzRJVnd1eUZv?= =?utf-8?B?dzBvdjJVUlBDUTZuejlnOTBodXRoai9WN29ZSVpRN2FmZGdjMTRBYTJwK2xR?= =?utf-8?B?U1NIRHRNV2c0MjU1a3lmdEtrSEdlOGsyZHJyUExYZkVZY1lIL3NYMXhoU1c1?= =?utf-8?B?WGZaWGVjS2pkUStQcjExUTFOSXRrSURKbW1UV2xmeHNZU25VZGliUGtabDZR?= =?utf-8?B?YjYzWnZWd0ovbXNOY0ZDU0RsMTNBbTVVSWpIck1LcWtBZndCQU9tVWxCdGNl?= =?utf-8?B?V3BGTDhESGxwdmwrcXRRakFBT1hRaGVNaHJJenZ3Nm94ZWJjZGt3RXFYR3FZ?= =?utf-8?B?NFZxU05RVlFGNHN6RlRiT1h3SkxzRUd1aFB1VTNMZmIxTDBjL0R5Tkl3emcr?= =?utf-8?B?UjdOZXRXc1ltTU5jaDZOYlJ4VzNSM05mYUlUKzE0MlZUWEhzcVpzRCs3Unhs?= =?utf-8?B?RGRrTE9ZUjlSS3BTbTJ6Y2NRTFVhNlUvcCt5UVh6U0F0ajVjNHhtY1krSXVt?= =?utf-8?B?U2NIQjZ2VEQza1dGL1hIbEVKQWNjWUg1OHAvWkZhSTJQd1hSNlVmWkFZL2hI?= =?utf-8?B?NUhCcXJlQ25ieUpsMUtiR21ibHRTSVlnYlI5OFppbnZJSVZVY251eFlYWCtX?= =?utf-8?B?MTlIc3dlUTFZT0tTQ09Uek01dXlLTm80Rzc5d3I2dFl5cS9NM1VzTDNRZE1j?= =?utf-8?B?L0ZsT1ZUeEJva0UzdVFqdGsvUVlRQ1BiSU9YWkFkS0pLM3RqdjJ0Ri92c3Bo?= =?utf-8?B?eHc4YityOHJsNG1YbWFYZEdkeWNHU2taTXZkNFRvUzB6ZkFQMVk3MWY5VktC?= =?utf-8?B?T1BtbzgxRHhqWTRZc09LUFRyVW1DZk5NNktRK1h3QWZpZnBmVndxUzVKemxR?= =?utf-8?B?WlI4ZFEwQXE5ekZhZnJyc2paZVpIZG1rczMwRUl4ckdNTkNsU1duL1loRGJr?= =?utf-8?B?QUpMUkk0M1lKc3FVbTJuV2R1ZU1nT1NNZ09TcjlxQmN4MEVXdWJqR3lMRi9L?= =?utf-8?B?L0xHbHhITTBhVzl3MXNRcGZDL3FMOTQwR0FLanNVNXR0dlFwd3QwUkdFbGRj?= =?utf-8?B?ZFUvSzE0QTAvdThZbmFJMWF2c1BBQUdkaVhGb1MrSlU4WnZyem1lUVNxQWho?= =?utf-8?B?WTh6dE5xUU5VL3QrUFg0eGJwaTFIVGp1SW9XUk41VE0xNzEyNmcxV0NCU0sv?= =?utf-8?B?Zy9TQlVPaFpBZDNjWWxMMmg3NTg0a2RxZ2l4bDNKZndTOEcrSDF5anJERjNL?= =?utf-8?B?SWp6SFJmSm41WTVyUmh1MER6emd3ZGpkSGxwVUdaU0hpYTFRSkdKdEM4NmtO?= =?utf-8?B?RDlYZTJxbHd0aGhmTHRtdHJXYk5FYStNRWk3RVZGUlJGQTlkaDl1Sm92ZjVI?= =?utf-8?B?MGxpOWNRQ1A0SkgxeVV0WGw2OGV0ZkpSZlRzNndVNjFMQlVqRm1jcDg1RE1C?= =?utf-8?B?eFdNNWhud1NnWkZjZm12bWl1cythcVQ4RmJucXdLZmRDYUltaXNEU0xhemQ5?= =?utf-8?B?UHBGeDdjWXZnd1I2N2RoWlRmZFRhcXhkeElxSDJ3dmRIYjJSWGRmM3ZkUWgz?= =?utf-8?B?ZDBKWmVkNEw2ZGF5UDFxa1Z1eE80VzNNc3dJK1J6ZUNtMVkrOUJwTDBqaTBF?= =?utf-8?Q?U4n0cLTn23wkqI8GcMzgqvI=3D?= X-OriginatorOrg: suse.com X-MS-Exchange-CrossTenant-Network-Message-Id: bf198489-fb39-4abd-d27c-08d9ab464c9b X-MS-Exchange-CrossTenant-AuthSource: VI1PR04MB5600.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 19 Nov 2021 10:21:07.2275 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: f7a17af6-1c5c-4a36-aa8b-f5be247aa4ba X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: WkmYN1gGkjSfwEywigRb5Pc1IamV2xsx92mtZ9ZlV83hSGrdEVkb9cNY3simB2NBxJkRVvukl5rsgISvWQ0N6g== X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR0402MB2704 X-ZohoMail-DKIM: pass (identity @suse.com) X-ZM-MESSAGEID: 1637317308585100001 Content-Type: text/plain; charset="utf-8" From: Lasse Collin It's good style. I was also told that GCC 7 is more strict and might give a warning when such comments are missing. Suggested-by: Andrei Borzenkov Signed-off-by: Lasse Collin [Linux commit: 5a244f48ecbbd03a11eb84819c5c599db81823ee] Signed-off-by: Jan Beulich Acked-by: Daniel Walker Acked-by: Julien Grall Reported-by: Yu Sun Reviewed-by: Luca Fancellu --- Linux has meanwhile further moved to using the "fallthrough" pseudo- keyword, but us doing so requires the tool stack to first make this available for use in at least stubdom builds. --- a/xen/common/xz/dec_stream.c +++ b/xen/common/xz/dec_stream.c @@ -583,6 +583,8 @@ static enum xz_ret __init dec_main(struc if (ret !=3D XZ_OK) return ret; =20 + /* Fall through */ + case SEQ_BLOCK_START: /* We need one byte of input to continue. */ if (b->in_pos =3D=3D b->in_size) @@ -606,6 +608,8 @@ static enum xz_ret __init dec_main(struc s->temp.pos =3D 0; s->sequence =3D SEQ_BLOCK_HEADER; =20 + /* Fall through */ + case SEQ_BLOCK_HEADER: if (!fill_temp(s, b)) return XZ_OK; @@ -616,6 +620,8 @@ static enum xz_ret __init dec_main(struc =20 s->sequence =3D SEQ_BLOCK_UNCOMPRESS; =20 + /* Fall through */ + case SEQ_BLOCK_UNCOMPRESS: ret =3D dec_block(s, b); if (ret !=3D XZ_STREAM_END) @@ -623,6 +629,8 @@ static enum xz_ret __init dec_main(struc =20 s->sequence =3D SEQ_BLOCK_PADDING; =20 + /* Fall through */ + case SEQ_BLOCK_PADDING: /* * Size of Compressed Data + Block Padding @@ -643,6 +651,8 @@ static enum xz_ret __init dec_main(struc =20 s->sequence =3D SEQ_BLOCK_CHECK; =20 + /* Fall through */ + case SEQ_BLOCK_CHECK: if (s->check_type =3D=3D XZ_CHECK_CRC32) { ret =3D crc32_validate(s, b); @@ -665,6 +675,8 @@ static enum xz_ret __init dec_main(struc =20 s->sequence =3D SEQ_INDEX_PADDING; =20 + /* Fall through */ + case SEQ_INDEX_PADDING: while ((s->index.size + (b->in_pos - s->in_start)) & 3) { @@ -687,6 +699,8 @@ static enum xz_ret __init dec_main(struc =20 s->sequence =3D SEQ_INDEX_CRC32; =20 + /* Fall through */ + case SEQ_INDEX_CRC32: ret =3D crc32_validate(s, b); if (ret !=3D XZ_STREAM_END) @@ -695,6 +709,8 @@ static enum xz_ret __init dec_main(struc s->temp.size =3D STREAM_HEADER_SIZE; s->sequence =3D SEQ_STREAM_FOOTER; =20 + /* Fall through */ + case SEQ_STREAM_FOOTER: if (!fill_temp(s, b)) return XZ_OK; From nobody Fri Mar 29 08:33:21 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) client-ip=192.237.175.120; envelope-from=xen-devel-bounces@lists.xenproject.org; helo=lists.xenproject.org; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) smtp.mailfrom=xen-devel-bounces@lists.xenproject.org; arc=pass (i=1 dmarc=pass fromdomain=suse.com); dmarc=pass(p=quarantine dis=none) header.from=suse.com ARC-Seal: i=2; a=rsa-sha256; t=1637317327; cv=pass; d=zohomail.com; s=zohoarc; b=Oliqooa51zYEt64F+sOneEKZT8d95tAF/TpqBM14abFFmERHzdl5y7J88F+rqKmDAXDC842MvuLYcudATm+0Enm/fNbUufjDHVv21Y6sEO3gCXOSaPiV3/LdlH1M1F/wwRSMQnFy6ZhPKIDpv16qSUWvNE+8uKNyDuVW/LQvd4c= ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1637317327; h=Content-Type:Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:To; bh=GS19NKXjFTcuZWiTvSaFRadeqka4sVuIO4lh4DFHodI=; b=c5bmFoS8JaflKJPy5Vr3A7jZrt9lP2ssYSUd4cYG6XbJCUwTajzc8kOejno7kpXAkiY+7yKo21iiR4TKs+TPVX0pneIoGJR7mwSSQchmMReIxNTivJAj2E7YLt0+HQ4weV39KBPifva03eNTMKuzIKTzUvs6KFNvnIIm5ED99J8= ARC-Authentication-Results: i=2; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) smtp.mailfrom=xen-devel-bounces@lists.xenproject.org; arc=pass (i=1 dmarc=pass fromdomain=suse.com); dmarc=pass header.from= (p=quarantine dis=none) Return-Path: Received: from lists.xenproject.org (lists.xenproject.org [192.237.175.120]) by mx.zohomail.com with SMTPS id 1637317327961434.7707493796312; Fri, 19 Nov 2021 02:22:07 -0800 (PST) Received: from list by lists.xenproject.org with outflank-mailman.227723.393969 (Exim 4.92) (envelope-from ) id 1mo11R-0002Pk-8d; Fri, 19 Nov 2021 10:21:33 +0000 Received: by outflank-mailman (output) from mailman id 227723.393969; Fri, 19 Nov 2021 10:21:33 +0000 Received: from localhost ([127.0.0.1] helo=lists.xenproject.org) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1mo11R-0002Pb-5Y; Fri, 19 Nov 2021 10:21:33 +0000 Received: by outflank-mailman (input) for mailman id 227723; Fri, 19 Nov 2021 10:21:32 +0000 Received: from se1-gles-sth1-in.inumbo.com ([159.253.27.254] helo=se1-gles-sth1.inumbo.com) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1mo11P-0002Lq-Un for xen-devel@lists.xenproject.org; Fri, 19 Nov 2021 10:21:32 +0000 Received: from de-smtp-delivery-102.mimecast.com (de-smtp-delivery-102.mimecast.com [194.104.109.102]) by se1-gles-sth1.inumbo.com (Halon) with ESMTPS id 76ae808c-4922-11ec-9787-a32c541c8605; Fri, 19 Nov 2021 11:21:31 +0100 (CET) Received: from EUR02-AM5-obe.outbound.protection.outlook.com (mail-am5eur02lp2059.outbound.protection.outlook.com [104.47.4.59]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id de-mta-23-QvYLr_5vNNaLS5_My8ecOw-1; Fri, 19 Nov 2021 11:21:29 +0100 Received: from VI1PR04MB5600.eurprd04.prod.outlook.com (2603:10a6:803:e7::16) by VI1PR0402MB2704.eurprd04.prod.outlook.com (2603:10a6:800:b5::13) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4713.22; Fri, 19 Nov 2021 10:21:28 +0000 Received: from VI1PR04MB5600.eurprd04.prod.outlook.com ([fe80::8062:d7cb:ca45:1898]) by VI1PR04MB5600.eurprd04.prod.outlook.com ([fe80::8062:d7cb:ca45:1898%3]) with mapi id 15.20.4713.022; Fri, 19 Nov 2021 10:21:28 +0000 Received: from [10.156.60.236] (37.24.206.209) by AS9PR06CA0420.eurprd06.prod.outlook.com (2603:10a6:20b:461::7) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4713.20 via Frontend Transport; Fri, 19 Nov 2021 10:21:28 +0000 X-Outflank-Mailman: Message body and most headers restored to incoming version X-BeenThere: xen-devel@lists.xenproject.org List-Id: Xen developer discussion List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Errors-To: xen-devel-bounces@lists.xenproject.org Precedence: list Sender: "Xen-devel" X-Inumbo-ID: 76ae808c-4922-11ec-9787-a32c541c8605 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.com; s=mimecast20200619; t=1637317291; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=GS19NKXjFTcuZWiTvSaFRadeqka4sVuIO4lh4DFHodI=; b=WDULcfIX7s+s8mfgJmXGMbAPvzGdXbXOz4V8EA/SaZeUWHSX9HHrtOIZ5Rmr5vZkWMTz55 HMWEpEwCnnPZWYrLJgyit3nwRpIN3xsb2ficzei5I+1jeAvCRnYOSVnuZSaFsjURkqs6ne DXxHn6qszt41tglEtLtOIYPMwyK5Uas= X-MC-Unique: QvYLr_5vNNaLS5_My8ecOw-1 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=WUXb9tLPbIsfNn140TXGHMGqER8liqGI+RmeDyfAHjDJAppJ2KiTyuuwDAF3CR0mjTgoSnU2KNZaglaJPgJmtgE5DFl4O3TlYSM/M0uxgkugGy1zvQWfhuZ6+iQ/w2xPsi9FObyNcYOV+MZ4e/yRTpWPUpCqOx+Lf2m5N10mrLzbS/uWXkxdeDoJPec5Z9VP+Tj9bS8EvEw6iv87O0e+ax9uzQWsLBVsNA5wOdalzrRw3k4vmkhZDgnThJ8nT4Jr4QWJJESh467y2O5MmOZUC3M8gSJVMu6AVIedVNPz1nX+n5E26m1AgeKmZePJ6f8T7ovCYrhZmvJvC8PLFcd8WQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=GS19NKXjFTcuZWiTvSaFRadeqka4sVuIO4lh4DFHodI=; b=gLqI3Lz/XnSKvqwCMngZTdz0yCtsBcIkYt2F7K9Prol//aFssdfA8AgR6lcACNfnDAN7n5XU4lOY/J4H95JZjm9e207DQZJUjPZFzC7aLpK0NkoGlSkmLcL/SRYkvRlhz9L+6Ko4nVghfO5Q+5XeLfiXJs7ex4OBBCmwCqQGOL9QmaKWXWCxK7C6o1bytPlE/ZD6ECojqlo93SejvpA7MB2y88HZXwUw8UjYMtLdmbATHipE8fvH1s2dfoRfiADpm3f0jn3BUofx2rNBZQ0oWEPoYpgRaKUrrPX5TTkApVv9y9GGWqNl+6MEBPeyWT20wJBpTi/ipJ7YnK75p0cfwA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=suse.com; dmarc=pass action=none header.from=suse.com; dkim=pass header.d=suse.com; arc=none Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=suse.com; Message-ID: <196008a7-ce37-52bf-13d3-4a08f203957a@suse.com> Date: Fri, 19 Nov 2021 11:21:26 +0100 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:91.0) Gecko/20100101 Thunderbird/91.3.0 Subject: [PATCH 2/7] xz: fix XZ_DYNALLOC to avoid useless memory reallocations Content-Language: en-US From: Jan Beulich To: "xen-devel@lists.xenproject.org" Cc: Andrew Cooper , George Dunlap , Ian Jackson , Julien Grall , Stefano Stabellini , Wei Liu References: In-Reply-To: Content-Transfer-Encoding: quoted-printable X-ClientProxiedBy: AS9PR06CA0420.eurprd06.prod.outlook.com (2603:10a6:20b:461::7) To VI1PR04MB5600.eurprd04.prod.outlook.com (2603:10a6:803:e7::16) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 632158c3-9aab-4687-a52d-08d9ab46595d X-MS-TrafficTypeDiagnostic: VI1PR0402MB2704: X-Microsoft-Antispam-PRVS: X-MS-Oob-TLC-OOBClassifiers: OLM:56; X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: Gwkxk01Ge/bdJr/rCEKnQBx8AkdIrY55WVzlQNN2aDATXV9d6ylbZySjgAhFuVMO+NewmFm5pSjQ9JHUDDfzaILgLVBXZwKHg/XbNujkoZ+VxnYxSSABd76OssXt7toUB83pALFUHWZivHpLvJtHZC/N4+1MWsZ58CQFEU4s0irnmTdBJf9AbBzfZqfYMDqBF++hBNivSAV+qZX2Uh4o+qjkoO2uAEogWyczD8JrPpUQdSlpCsqb3fWl7RAmXfk2r4hVbv8RaJ0/RpH03+dvWJaawhpbPTC7Uf5rbyHcN+mtJsxy950ZjV+CZkX9T2/Rpt4sPwCYA0kkCd15/FZpfIzvo6qN9ApfkI6SBOWigVRDolRRo1StTofnaScuudo9pG+lhSEApP+wbfgMQaYnzJTjjSM22fcyjOu9Fqohw4YxN8XisCeDNtX1fbGBeoWuO8zCxn6xiH4aD5bwB6o2ujI2ElQLmSkQ12zWSYpsvcirWn5WUphpK5Jd4g26EsT8fX0A5K0YoolprypR+qErPKSoGBPeY26TklVkwgPAfvaJjcSR1CaRc89OSDPCcUuNerLvoZTII1VKNCs8/jxQELywDjDgOh9UFKZKc744ahw3OHflN5GKxTzDTZiuEzR1JLg9NXXCK4tcr2OWPnH4XDWDgUSRfTUcoaLuj1O+1N0lrz8o9JkUpIohHt2FEc4uAM4heRis6h6kUrvtQhvLNdy6WvcJ6GFzc4sLCUscs5as9QMYRN6aLXnzopBrdUKh18U1dB3Hmd2k8TzdgyvM3uXa5/3WqkDsJY26FbBIjavyvdoDBgWk+a/DzLj7nSzs X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:VI1PR04MB5600.eurprd04.prod.outlook.com;PTR:;CAT:NONE;SFS:(366004)(31696002)(38100700002)(36756003)(508600001)(2906002)(31686004)(8676002)(66476007)(66556008)(86362001)(956004)(2616005)(4744005)(5660300002)(26005)(8936002)(6486002)(186003)(316002)(966005)(66946007)(54906003)(6916009)(16576012)(4326008)(45980500001)(43740500002);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?Ukluc1JCUlAzUHcrWE8wbHVsOEZmTVBhbGxicjZzdldzNDVPWXFaVHFYdVlS?= =?utf-8?B?ODdjbktrWjc4TFkrQUs1VWFINHNTT3JqY0EraFhsT3gyY3dZWll6WjVUeG5H?= =?utf-8?B?dkRVZ0hSMGFXQjJrdzBPbGY3d0hsM3RZdG5LNHAyY2tiV2QyOFZ0d2pIUkI0?= =?utf-8?B?ZGRPejE5eFliTjk1VE5Xc2RzcEZiS3VMYk93N3pzNUJqWVVXNGNBRnRoK0N2?= =?utf-8?B?cnR6eVVESjBuRE0yeEVyOEQvbkx1dG1SRmdJeksyc1ltMUNsclI3NlBDcFVI?= =?utf-8?B?MmVpNWRQUHcvTTVrMzZxSnBSWERxZjRxOVpJSmJLQTFiZ3RORW50QllNM2l0?= =?utf-8?B?TUZGaFlCSFNxV0dmV2FvNDBqTStLZjFKSnhvU09mYytMSE1yak5qUmloQi8w?= =?utf-8?B?aHZFeXoyQTg0c3RNMDkvUWlYNm1DZkpyelJKVmFvdkN6eHJFZVk4TVhBU29z?= =?utf-8?B?OWthZkxLMmExdWhnRVpWWkwxTThwaFJPY2ozN1BuRnJLS2hDb05oSlhxc254?= =?utf-8?B?MkpKZGp1Z254N3BPbWEwSDJ6dzhZcGt1MiszcFZQZVAxRmh2S2Ntdm9ZUUR1?= =?utf-8?B?N1VWYU5xd05oUy9DTW80Tk1DSmVYMzYwbEFubFo3Q3pzREdWeUU1VzJhN1Qy?= =?utf-8?B?QnZ0cStFTkRLUnpDTjhlRVBIU2RLZEJoN0YyNENBek5DVENvMkhKR1JadlVw?= =?utf-8?B?dEJraE1DeDFqSE1IODhjRklxMFlXdTlpa01sQUZJdHVFMU9veHUwdHFVb09E?= =?utf-8?B?NVZaTm5kb09ROFVpYjZ4bnpQV2FDR0ZvZk80MVg1bGNnVWVadTVhUW5Vazg5?= =?utf-8?B?RTZCUDcxd1hMeU03cWlIQXFYdCtpd1hIY0VRaFRtK2hIY0p3L2k4WEFzaUMx?= =?utf-8?B?QmNlTWdWSHliV2NiRURYTUhUUWVqUnhKcCtLb05PR0dxOEtJcmRBWHVvcjZT?= =?utf-8?B?d3pNVHYvRit6emFIRDFwclkzNG5xcXRDYkVZN2VQM2dIa3lja1Mza0R1YXFz?= =?utf-8?B?TlJPUVFGY3RNNVk5K2R0UGJGZ3plVHFjVytQOGJseWUyN1hHZUZuZldwZC9q?= =?utf-8?B?N1BjdzhFRU1QSnA1OThvb0lQeGZHcjNBVDhpcFIvbVhJMHRnT2phUDJWU01R?= =?utf-8?B?SXhuLzNBSWZhc2lrRE9kQm1ZVjZzZ2hsNWRJQ2dPblRwRGJ0ejJJQ0xraHJH?= =?utf-8?B?YVJkUWVQWmNRRWo3alZ4SzVsQVNjU25YZmMzaTVqZklqeU1RUC96Tk56S0hP?= =?utf-8?B?OUpYVnczZVVsL1VCQ3l1TXRUa2EvTEUrVjAyQ1lKWnRVZG0zN2FQcENLMHps?= =?utf-8?B?Uy9lbnM2bDV2WHQvZlJJWDE5UDh6TzZ2czkwZVNTQVNRbEMzMzRLdGQrd3dR?= =?utf-8?B?QTd6dm1FTlRwU2VDcFNDZVI0R1ZBNHFlT0xZTWRib2lNbFdQWlBvSUJESDM3?= =?utf-8?B?R2JpK0JXNzBGNThjSWZRSlVjWVJMTmk0WDh4RWJCYm40M3AyN1c0S0l2aklZ?= =?utf-8?B?RGFmRHBJLytYU2szOXc3YU1GMmxCc0FnRmhjOXREcEw5RVlQS0hHY1c5ZjFS?= =?utf-8?B?YlJDKytMMU0vVFpCV0FQT1JNTDhIRDBtN0YyZUhGRmJFdTR6aXo0ZmZ2akIv?= =?utf-8?B?ZTV2ZGpENThGczFWbkZoSWRuZHhKTlBvTWw5T2JWNjNGZlZTM1BaMHZtdmxm?= =?utf-8?B?cUE0akN3UlNmRU1aVUF5dkRGV04wSWZOQ3EyOUd5ZGJRVGlxRkUyS01GWFg2?= =?utf-8?B?bkZlVnRvUWhwTmsxT0x5eGQ1bTVuUEJockZSOGt0Yk5tWjlqb0IyWXlpYzVu?= =?utf-8?B?cWlKTmMyVWhqNXBYR1lmTjlFVXNwTGtZcFhiSDRtdC9zVEM0aWY2b1Q0TzlI?= =?utf-8?B?cE4zaTM3RTRxWjB3OSt5emlOdTdLSE9CQWxJeEhhVFFGUmFHL29yWWxQTW9y?= =?utf-8?B?K25TUllESnYzWWZLeXBYUWJVVGVtSm9JSFBPKzZ5cGJrYWVkTk9TNjZDWTJN?= =?utf-8?B?MEdzYUtuV3JZZnE0Q0NpOU10NDIyVWhnU0l4WnVJWXE4TEVwYU9wV1ZrVmth?= =?utf-8?B?U282Y3N5NWZPbFJ5WUFvUDlUM3M5a3ZSb1Q5RFFTZUxYbEQ4UVlmVkFqalN2?= =?utf-8?B?UVE1U1kxSW91TDNsWWR3TGNFOXJkeGdPZ29SUnZvbHZYTEdrekZ3a21RV3ZL?= =?utf-8?Q?jIFaJj5OK2Q+QIMjeg2zpqg=3D?= X-OriginatorOrg: suse.com X-MS-Exchange-CrossTenant-Network-Message-Id: 632158c3-9aab-4687-a52d-08d9ab46595d X-MS-Exchange-CrossTenant-AuthSource: VI1PR04MB5600.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 19 Nov 2021 10:21:28.5895 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: f7a17af6-1c5c-4a36-aa8b-f5be247aa4ba X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: ks5r58P7QlroyeKlAEsnhUqMs6WBR3xJJFvZqdEiGct38Yn1wlmj2GOK9uPdcb3XtgIWYEKgd0wvCsVoTt1+Ow== X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR0402MB2704 X-ZohoMail-DKIM: pass (identity @suse.com) X-ZM-MESSAGEID: 1637317328299100001 Content-Type: text/plain; charset="utf-8" From: Lasse Collin s->dict.allocated was initialized to 0 but never set after a successful allocation, thus the code always thought that the dictionary buffer has to be reallocated. Link: http://lkml.kernel.org/r/20191104185107.3b6330df@tukaani.org Reported-by: Yu Sun Signed-off-by: Lasse Collin Acked-by: Daniel Walker [Linux commit: 8e20ba2e53fc6198cbfbcc700e9f884157052a8d] Signed-off-by: Jan Beulich Reviewed-by: Luca Fancellu --- a/xen/common/xz/dec_lzma2.c +++ b/xen/common/xz/dec_lzma2.c @@ -1146,6 +1146,7 @@ XZ_EXTERN enum xz_ret __init xz_dec_lzma =20 if (DEC_IS_DYNALLOC(s->dict.mode)) { if (s->dict.allocated < s->dict.size) { + s->dict.allocated =3D s->dict.size; large_free(s->dict.buf); s->dict.buf =3D large_malloc(s->dict.size); if (s->dict.buf =3D=3D NULL) { From nobody Fri Mar 29 08:33:21 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) client-ip=192.237.175.120; envelope-from=xen-devel-bounces@lists.xenproject.org; helo=lists.xenproject.org; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) smtp.mailfrom=xen-devel-bounces@lists.xenproject.org; arc=pass (i=1 dmarc=pass fromdomain=suse.com); dmarc=pass(p=quarantine dis=none) header.from=suse.com ARC-Seal: i=2; a=rsa-sha256; t=1637317356; cv=pass; d=zohomail.com; s=zohoarc; b=Ukzd1eYd/GGROvbTFZuih1JISHn1gk5HMaSYKElTiYVvirXmWnOYYclekjCGwelZMUhp2GRvHLNu4SluRhuntoaB+/mT+XggY0/SL3hx6Jq4H9ToitZBsuDelfRrX1U8F/qtPo+nznsxA+Bz8kZ4UkB9VjrQgK+i34DLwUmADq8= ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1637317356; h=Content-Type:Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:To; bh=Qo4aw9jO+x/yjshebq2fIVWJsdP1zKWvaVAQTszqXLM=; b=Q17/tfDydI8S5euQD2f5s3kalg3cKs6BvxoJx4auVsI2O1tLssbo1SOlRaKX7pbtqYXi3/Dl4/y6Rn/vg91yHf5jj7fUEJbTHakmS6VlwkjxZjGC2HO9Zgq4XvpWCdZFwxr4jO9mq+Y3t69hRD3o1335cqktp2UXeQx+ko3hPrs= ARC-Authentication-Results: i=2; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) smtp.mailfrom=xen-devel-bounces@lists.xenproject.org; arc=pass (i=1 dmarc=pass fromdomain=suse.com); dmarc=pass header.from= (p=quarantine dis=none) Return-Path: Received: from lists.xenproject.org (lists.xenproject.org [192.237.175.120]) by mx.zohomail.com with SMTPS id 163731735693913.983039615491407; Fri, 19 Nov 2021 02:22:36 -0800 (PST) Received: from list by lists.xenproject.org with outflank-mailman.227726.393981 (Exim 4.92) (envelope-from ) id 1mo11s-00033t-HO; Fri, 19 Nov 2021 10:22:00 +0000 Received: by outflank-mailman (output) from mailman id 227726.393981; Fri, 19 Nov 2021 10:22:00 +0000 Received: from localhost ([127.0.0.1] helo=lists.xenproject.org) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1mo11s-00033m-EH; Fri, 19 Nov 2021 10:22:00 +0000 Received: by outflank-mailman (input) for mailman id 227726; Fri, 19 Nov 2021 10:21:59 +0000 Received: from se1-gles-sth1-in.inumbo.com ([159.253.27.254] helo=se1-gles-sth1.inumbo.com) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1mo11q-0002Lq-Q9 for xen-devel@lists.xenproject.org; Fri, 19 Nov 2021 10:21:59 +0000 Received: from de-smtp-delivery-102.mimecast.com (de-smtp-delivery-102.mimecast.com [194.104.109.102]) by se1-gles-sth1.inumbo.com (Halon) with ESMTPS id 86a1f2e4-4922-11ec-9787-a32c541c8605; Fri, 19 Nov 2021 11:21:58 +0100 (CET) Received: from EUR02-AM5-obe.outbound.protection.outlook.com (mail-am5eur02lp2058.outbound.protection.outlook.com [104.47.4.58]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id de-mta-9-ZVJ6HCxxMOOcOERfDptcKw-1; Fri, 19 Nov 2021 11:21:56 +0100 Received: from VI1PR04MB5600.eurprd04.prod.outlook.com (2603:10a6:803:e7::16) by VI1PR0402MB2704.eurprd04.prod.outlook.com (2603:10a6:800:b5::13) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4713.22; Fri, 19 Nov 2021 10:21:55 +0000 Received: from VI1PR04MB5600.eurprd04.prod.outlook.com ([fe80::8062:d7cb:ca45:1898]) by VI1PR04MB5600.eurprd04.prod.outlook.com ([fe80::8062:d7cb:ca45:1898%3]) with mapi id 15.20.4713.022; Fri, 19 Nov 2021 10:21:55 +0000 Received: from [10.156.60.236] (37.24.206.209) by AS8P189CA0002.EURP189.PROD.OUTLOOK.COM (2603:10a6:20b:31f::21) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4713.21 via Frontend Transport; Fri, 19 Nov 2021 10:21:55 +0000 X-Outflank-Mailman: Message body and most headers restored to incoming version X-BeenThere: xen-devel@lists.xenproject.org List-Id: Xen developer discussion List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Errors-To: xen-devel-bounces@lists.xenproject.org Precedence: list Sender: "Xen-devel" X-Inumbo-ID: 86a1f2e4-4922-11ec-9787-a32c541c8605 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.com; s=mimecast20200619; t=1637317317; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=Qo4aw9jO+x/yjshebq2fIVWJsdP1zKWvaVAQTszqXLM=; b=RYqw2J0yOtH3His1mRNLG9dG7K/4vIT71hJH5TLWQOaYgQXvR/sTgIXdEUZ7aL/l+ARMmg NsRgIytXcaXv2fTVZzZAY1l2wht6gPfkCAI2MOrJjEZUdAAdbT9Rn78KbBfgRbtIgtlXQd 7y18YCxo8Ttqc0XQfMxlSY5GXKKAEzU= X-MC-Unique: ZVJ6HCxxMOOcOERfDptcKw-1 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=E/TWJ94NsJ4TYyaR7dEgP0JAbgm6tGbhLaE6HrDAIjJ8cXAa9VhTVTjQa0EOo5r/1sgdOUC3y/Be6LjUIocWYSdGc+KCpQWMHjp58s8/FY0fpIlJzqIPoTYx6trnLmj5KIRFRDnHJSbxEHYzbuEvSC7A0I0YYuydleq/ist51M1NYdoUxDySbP7/Xold+rf2W3x/Xnnii7YpNzncC8j2x2mfXgedPsHeIqndOsUNQLNSNGQECvgGJ50SvzdPbhKVWmbrNdV4g9leKZFw3Amihh1Ln9ghpX6LqpHZhKyaXSj0GZUgmbe8U6AsXUX8ADv3DfmgBqbUCNeNSWyU58i7oA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=Qo4aw9jO+x/yjshebq2fIVWJsdP1zKWvaVAQTszqXLM=; b=B+ehqTDJVyejfUQ4kdAuqaupy8w1o/E/v4gYqlb1IoN2rGLiNbky6s4pZFCvSwxFs9wIwyEAOLIgftsKwtvrUq6OY1LBVBaYzJfGtHu5CLvZ6zkci44e9/j++8yw1EDShtHqoFaqRKcazdpUy4cMCwBP7VtFcA28NqKLbQ5voqF4UlNEiYaCqlnbYyRwNHjAIlkjq8lfc4+Pc/jTtP9TBLf8D/oZaDC6K9OGKCSZVPiOJNJK8xebGs7lizlsWcZDdALDveTv3wz+Dxezc8SiYFNizW6+FSmEnC6IEUccTGwan7Wqdj8wIzsOzTXHRTnrbVguVV+iKnRUv7JWUSfd9Q== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=suse.com; dmarc=pass action=none header.from=suse.com; dkim=pass header.d=suse.com; arc=none Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=suse.com; Message-ID: Date: Fri, 19 Nov 2021 11:21:54 +0100 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:91.0) Gecko/20100101 Thunderbird/91.3.0 Subject: [PATCH 3/7] decompressors: fix spelling mistakes Content-Language: en-US From: Jan Beulich To: "xen-devel@lists.xenproject.org" Cc: Andrew Cooper , George Dunlap , Ian Jackson , Julien Grall , Stefano Stabellini , Wei Liu References: In-Reply-To: Content-Transfer-Encoding: quoted-printable X-ClientProxiedBy: AS8P189CA0002.EURP189.PROD.OUTLOOK.COM (2603:10a6:20b:31f::21) To VI1PR04MB5600.eurprd04.prod.outlook.com (2603:10a6:803:e7::16) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 7d1c5f9e-809c-4df3-2c28-08d9ab466975 X-MS-TrafficTypeDiagnostic: VI1PR0402MB2704: X-Microsoft-Antispam-PRVS: X-MS-Oob-TLC-OOBClassifiers: OLM:335; X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: ovbjpF5fkrJ5cw1kCDFH4WWahZktHL00tWGoUMXK7xRFfMPxSu/aZ6S+qZT71EZ6dxA9yKrHCuxe4t1yHFuxIhdnFxT3C+XRUIicie3oLuFGQSCPNMK6hKx/XPUMQ3xUe/nYiUGaPk1R4lZ33qCEDRp1Z67VuZQEDfkngXr5cGbPLKqNcezWwP3dI2JPp1THA8NAXfUMwUSh90paMT5UYhBGuRMIZE63Jywa4Vu7PrPpqz/ZE1kflSfXgvxBiAtb4S3nlPRydux3FXgbRN7XTJM5pQGgWTTrPSanrmhJIWduUrQaZj9QdJstvlBic0SAIui3QSG0tXGLEGtcHpscFFYGgL4U27jNcYdauz2zC3jjgyQ80fvSpgYQimmOnbt+uyFtJljhOgITFvzXoB3tWKzEx4Szr8zTfsgzLeT2zKsU2Vjfm/0mgjqa5xIcs5ECUgxNGwTTA23u9IkXlednTHssFwjcLIyNkHCiSbup/WIfRkfd73O/nY9oC9A5g9/tyipXW23zOU9eF19T2XkZKpNEJbIK89lznjH6UgxC0qaLnPp5P8KXxuzkbtESQ239B09q7krN8Askzt9saNpsjWBOU/mGBvHDBBfHoAfm3M5NAhpKe6cW9+oyseYDKrG/gtP80xl1vbVL1uiMGxG3mUaZh8jpYy0bx+sN/qIRIqXWZsE22K+mthw39YwrzAohTOIU9EUPI/vi9s7CYGnYZw2IPWd6HG2HOwAEVDzbPNBKNJtU3B0O30VIwebFLrYofaq+vFio3O82D0/GKSp0VcFPIYCIzaZiZ1FIpJvz/+IwPq1CnwbLXMgvB6E0pfE3APafg0P2AcTjusuWbsDCcw== X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:VI1PR04MB5600.eurprd04.prod.outlook.com;PTR:;CAT:NONE;SFS:(366004)(31696002)(38100700002)(36756003)(508600001)(2906002)(31686004)(8676002)(83380400001)(66476007)(66556008)(86362001)(956004)(2616005)(5660300002)(26005)(8936002)(6486002)(186003)(316002)(966005)(66946007)(54906003)(6916009)(16576012)(4326008)(45980500001)(43740500002);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?cE1pKzlwbVVIeDRQd2hBbWlqYjA3RkZvcnRFMUFnSFdWU3BESFNleXJlR0lX?= =?utf-8?B?L0VyR1N6Y2hISUhpbitNVGlVWDFINThlbkR1UmJKMnZXeWFPNEord1ZxYktq?= =?utf-8?B?eG55Tk1UREduSVpDYVRxTzFVOGRYelhmeVVJUEtES1M3WFY4UW1BSThwVjg1?= =?utf-8?B?cSszV1FwNmZDYVFqbUVYdklBV01ybU9oNUFBdkRoVHY3QS9MUjhldTl5V3dG?= =?utf-8?B?RTVQNElJZnF4cWlaOXpPQkRhaVBWTmZleW5WREllTlNMRXZjNHpBMURoSkhY?= =?utf-8?B?ZWtHQmNTeVhnSlpvMzBlMGkzMStjZWFVS0R3L0s5UDhHSTR2bFhUQThwRDJ0?= =?utf-8?B?emNkN1BiSmN0L2t1UzdyZVFhNGpqR3BOb0lBNHppWVhnTWNIdXRjelpRdjBi?= =?utf-8?B?bWZuaTllN0J5dWNvR2ZGZWZ6OTlaVjRQTVhQUDFxbG5tMWVyVmdYdzdxZEVV?= =?utf-8?B?ZENXQ0FpQnp2VXNrb3VMcmlkQVdFY25RVStXV0w2Q2RJSGgyQlBRRTgzNDdO?= =?utf-8?B?L0x2YWpmR051bjFBSGRXbkl2SVFQUW5QbnJRelh0d1JaSkd2cDg1TmlkN3lF?= =?utf-8?B?RVNybkR4ZFBRV20xWjV3UDVXNjVackdIeCtBM0FwY29ydjUyd1laa2U1OFlU?= =?utf-8?B?T21uTys1T05pUVhmNzFMTDRCWjhoNXdOUm93Qm9ydW5LcVd4L1pMbWgvV2w1?= =?utf-8?B?dVJhbGhNYWJwU1h3aHBpYkFiZGp6blZWMElxQ2lUQVZsWFNFcWUvbVF5b3Nl?= =?utf-8?B?NENRNkoyOUV1UmprUVZINEVRQlV2N1JSR1F4ZzhtRWJrZk50T3g3Qjd3UlJ2?= =?utf-8?B?K2ZwcWJyV0gwRlhoYytKdUFlTUVtekU3dWpqSGpYQW9Ld0tKWjhDdWpId3Vu?= =?utf-8?B?cGVGSXpDY0t3cDIyTmJFWC9XeFZDOEgyaTdoQVY0bEJocHY4UXJ1MlNkRkhj?= =?utf-8?B?U01jdFlsd3htZjBsd2hmeitVbW5PS1MwRWZEcGJOSE91TTZyVmlQZ0pKdjhK?= =?utf-8?B?Sm9wak5KZ1dQRXM3MUc3TUMxOUNMQUdPSXNmU2JRUjFKR1VKTWNBZGFiblZD?= =?utf-8?B?aFdxNGQvVitzelA0NmFUbUhKaFUxQzFiaGI1L0RSalF1OTJWazBCZzF4cVdY?= =?utf-8?B?bDJpaE9zTXpaa1VZYXF2S2hpa2RoMWhRUkUyOElGWkp6bGZSKyt1VldSRWhj?= =?utf-8?B?a0pGY3d4bDdpTXE0TThoWm0yNHVBM1ZCWDh1NTdkc0hBOGZkNTYvV05VMWhq?= =?utf-8?B?OXVsZ1c2enYxVkwvNDdRWVFUbEdja2NGWTBqYzRRNjZ6aUtLN0dlNUZRU3lH?= =?utf-8?B?SlY0SzNmKzBnRG9QVWwxSmYwRVFEcUs5bkJsOEhwK1M4ODZMajB3ajdZcy9C?= =?utf-8?B?MDVac3NLSmFySDJtMUcyd2pMdkpSVVh3aDBXVXU1cmZHd1pVaWRZWmhzMVk4?= =?utf-8?B?Zy8yQXBxV3JRTElxMVZkWXBGOGFtbnBpWm16azl0dFpkcG5kSjd5bnhpbWE4?= =?utf-8?B?QndRNDVvQUs2M2l4bjNmMVV4MTFkRDFPSmRuRThGeEkrZjhUbE5PblBHbUk3?= =?utf-8?B?YUt1bm8zSVJOSHgxZEN3YUZqRVZBdDBmYTV5NzdQTzBIeDZpRk5taFBHV2Qy?= =?utf-8?B?WVNZQk92WEhtMmdQSHJITVBtVXpWNU1uMk10bmRsUXNReG4wN3RXdHlPNTE5?= =?utf-8?B?ZVpmbTRnL2hyalcxUGhFOG5NNm1MQ1diRnBwcjNOcGhIUS9nemkxemJvcFA5?= =?utf-8?B?VzNuaWprQjU1QjVSaXpkQVFZSzJDVGVxdlp0RXpub0JRRmtsZ2lNOVhWTzdq?= =?utf-8?B?YWpkaFRSVXU2NS9TZGtRa0VROFVYbVJkeGQ4MXVJbjlZL3pJekNmaFgrdHQ2?= =?utf-8?B?VjhVeithOXQzVTlSRlFJM3E1ZkdHb2IrK1V6UTBMQkFlWXdHVUtFN1pqTys0?= =?utf-8?B?UG1PY24ydVhNNFRmNEhPcEJ3WDhKWXVseDJ4QVRkTm4wQ005cVVUaXlxbko1?= =?utf-8?B?OWIybG1Cem5IZ2xiYzMxc1czU1VNUXhaam9ySHYydFpncDBSN0xjYXh6bW1U?= =?utf-8?B?MjJkN0tSbE0weEFPc01pTHFTRGlrcGdjc2cyUHNYMGhXbDdDSmNZeUhBb1FB?= =?utf-8?B?M1RhVGMzall6Nllkc2xzRmRJYmd6dkhEamZPZ1ZldFVWRGIzUmJVRmJWakls?= =?utf-8?Q?ervi4eTuKu/MiGGh9OnV5S8=3D?= X-OriginatorOrg: suse.com X-MS-Exchange-CrossTenant-Network-Message-Id: 7d1c5f9e-809c-4df3-2c28-08d9ab466975 X-MS-Exchange-CrossTenant-AuthSource: VI1PR04MB5600.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 19 Nov 2021 10:21:55.5922 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: f7a17af6-1c5c-4a36-aa8b-f5be247aa4ba X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: o5eL/MTSRGZg5mUvUl9UpQnX05S+kUQsrNGbvN1YoXmMuB4019jRC0awPRjooD4fBZx6jewnd6QYUErIfKH7Gg== X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR0402MB2704 X-ZohoMail-DKIM: pass (identity @suse.com) X-ZM-MESSAGEID: 1637317359032100001 Content-Type: text/plain; charset="utf-8" From: Zhen Lei Fix some spelling mistakes in comments: sentinal =3D=3D> sentinel compresed =3D=3D> compressed immediatelly =3D=3D> immediately dervied =3D=3D> derived splitted =3D=3D> split nore =3D=3D> not independed =3D=3D> independent asumed =3D=3D> assumed Link: https://lkml.kernel.org/r/20210604085656.12257-1-thunder.leizhen@huaw= ei.com Signed-off-by: Zhen Lei [Linux commit: 05911c5d964956442d17fe21db239de5a1dace4a] Signed-off-by: Jan Beulich Reviewed-by: Luca Fancellu --- a/xen/common/bunzip2.c +++ b/xen/common/bunzip2.c @@ -73,7 +73,7 @@ =20 /* This is what we know about each Huffman coding group */ struct group_data { - /* We have an extra slot at the end of limit[] for a sentinal value. */ + /* We have an extra slot at the end of limit[] for a sentinel value. */ int limit[MAX_HUFCODE_BITS+1]; int base[MAX_HUFCODE_BITS]; int permute[MAX_SYMBOLS]; @@ -326,7 +326,7 @@ static int __init get_next_block(struct pp <<=3D 1; base[i+1] =3D pp-(t +=3D temp[i]); } - limit[maxLen+1] =3D INT_MAX; /* Sentinal value for + limit[maxLen+1] =3D INT_MAX; /* Sentinel value for * reading next sym. */ limit[maxLen] =3D pp+temp[maxLen]-1; base[minLen] =3D 0; --- a/xen/common/unxz.c +++ b/xen/common/unxz.c @@ -23,7 +23,7 @@ * uncompressible. Thus, we must look for worst-case expansion when the * compressor is encoding uncompressible data. * - * The structure of the .xz file in case of a compresed kernel is as follo= ws. + * The structure of the .xz file in case of a compressed kernel is as foll= ows. * Sizes (as bytes) of the fields are in parenthesis. * * Stream Header (12) --- a/xen/common/unzstd.c +++ b/xen/common/unzstd.c @@ -16,7 +16,7 @@ * uncompressible. Thus, we must look for worst-case expansion when the * compressor is encoding uncompressible data. * - * The structure of the .zst file in case of a compresed kernel is as foll= ows. + * The structure of the .zst file in case of a compressed kernel is as fol= lows. * Maximum sizes (as bytes) of the fields are in parenthesis. * * Frame Header: (18) --- a/xen/common/xz/dec_bcj.c +++ b/xen/common/xz/dec_bcj.c @@ -422,7 +422,7 @@ XZ_EXTERN enum xz_ret __init xz_dec_bcj_ =20 /* * Flush pending already filtered data to the output buffer. Return - * immediatelly if we couldn't flush everything, or if the next + * immediately if we couldn't flush everything, or if the next * filter in the chain had already returned XZ_STREAM_END. */ if (s->temp.filtered > 0) { --- a/xen/common/xz/dec_lzma2.c +++ b/xen/common/xz/dec_lzma2.c @@ -147,8 +147,8 @@ struct lzma_dec { =20 /* * LZMA properties or related bit masks (number of literal - * context bits, a mask dervied from the number of literal - * position bits, and a mask dervied from the number + * context bits, a mask derived from the number of literal + * position bits, and a mask derived from the number * position bits) */ uint32_t lc; @@ -484,7 +484,7 @@ static always_inline void rc_normalize(s } =20 /* - * Decode one bit. In some versions, this function has been splitted in th= ree + * Decode one bit. In some versions, this function has been split in three * functions so that the compiler is supposed to be able to more easily av= oid * an extra branch. In this particular version of the LZMA decoder, this * doesn't seem to be a good idea (tested with GCC 3.3.6, 3.4.6, and 4.3.3 @@ -761,7 +761,7 @@ static bool_t __init lzma_main(struct xz } =20 /* - * Reset the LZMA decoder and range decoder state. Dictionary is nore reset + * Reset the LZMA decoder and range decoder state. Dictionary is not reset * here, because LZMA state may be reset without resetting the dictionary. */ static void __init lzma_reset(struct xz_dec_lzma2 *s) --- a/xen/common/zstd/huf.h +++ b/xen/common/zstd/huf.h @@ -131,7 +131,7 @@ typedef enum { HUF_repeat_none, /**< Cannot use the previous table */ HUF_repeat_check, /**< Can use the previous table but it must be checked.= Note : The previous table must have been constructed by HUF_compress{1, 4}X_repeat */ - HUF_repeat_valid /**< Can use the previous table and it is asumed to be = valid */ + HUF_repeat_valid /**< Can use the previous table and it is assumed to be= valid */ } HUF_repeat; /** HUF_compress4X_repeat() : * Same as HUF_compress4X_wksp(), but considers using hufTable if *repeat= !=3D HUF_repeat_none. From nobody Fri Mar 29 08:33:21 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) client-ip=192.237.175.120; envelope-from=xen-devel-bounces@lists.xenproject.org; helo=lists.xenproject.org; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) smtp.mailfrom=xen-devel-bounces@lists.xenproject.org; arc=pass (i=1 dmarc=pass fromdomain=suse.com); dmarc=pass(p=quarantine dis=none) header.from=suse.com ARC-Seal: i=2; a=rsa-sha256; t=1637317358; cv=pass; d=zohomail.com; s=zohoarc; b=H9CR9JmZ9ajCo6dRefclaJXbdLcIsnDtgflnKzRhOtZeImfsLcY/NaE1Xsfu4ulB1urZ2e/RnN1e1Jixo7M/Cwtn/LWYqWpqd5bkW83p5TizdaUmrdnUjuGAlnGBO+BnooyWmlDifAOubbIe4zbg95cEJxRjHccnwTRxYbbx64I= ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1637317358; h=Content-Type:Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:To; bh=Poc0wRx/mUi3cWDnszzqwQVxElnm2r9mLP8h58FAsVw=; b=TyL9y9rhOhay6Rjb0gIty9sjFYYTiWvrNGbdS0Wg9E2bd2xJBaDItSZN17OU2eILlvn/gKpor9piakbNvJ7tqBP8rHitiagFBrFuAUN7Q0QmG6bv327ZKxC9H8aKYwGN6Y/TVqVQzJ+JLzGbBSjnTg1jZLY2womk61RrYc+j8Ws= ARC-Authentication-Results: i=2; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) smtp.mailfrom=xen-devel-bounces@lists.xenproject.org; arc=pass (i=1 dmarc=pass fromdomain=suse.com); dmarc=pass header.from= (p=quarantine dis=none) Return-Path: Received: from lists.xenproject.org (lists.xenproject.org [192.237.175.120]) by mx.zohomail.com with SMTPS id 163731735873877.55668024294687; Fri, 19 Nov 2021 02:22:38 -0800 (PST) Received: from list by lists.xenproject.org with outflank-mailman.227734.393992 (Exim 4.92) (envelope-from ) id 1mo12F-0003db-Ri; Fri, 19 Nov 2021 10:22:23 +0000 Received: by outflank-mailman (output) from mailman id 227734.393992; Fri, 19 Nov 2021 10:22:23 +0000 Received: from localhost ([127.0.0.1] helo=lists.xenproject.org) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1mo12F-0003dU-Nl; Fri, 19 Nov 2021 10:22:23 +0000 Received: by outflank-mailman (input) for mailman id 227734; Fri, 19 Nov 2021 10:22:22 +0000 Received: from se1-gles-flk1-in.inumbo.com ([94.247.172.50] helo=se1-gles-flk1.inumbo.com) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1mo12E-0003FS-J0 for xen-devel@lists.xenproject.org; Fri, 19 Nov 2021 10:22:22 +0000 Received: from de-smtp-delivery-102.mimecast.com (de-smtp-delivery-102.mimecast.com [194.104.109.102]) by se1-gles-flk1.inumbo.com (Halon) with ESMTPS id 94d9d9ca-4922-11ec-a9d2-d9f7a1cc8784; Fri, 19 Nov 2021 11:22:21 +0100 (CET) Received: from EUR02-AM5-obe.outbound.protection.outlook.com (mail-am5eur02lp2051.outbound.protection.outlook.com [104.47.4.51]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id de-mta-17-AXYmrSAsPE6VtmYQLeNg8g-1; Fri, 19 Nov 2021 11:22:20 +0100 Received: from VI1PR04MB5600.eurprd04.prod.outlook.com (2603:10a6:803:e7::16) by VI1PR0402MB2704.eurprd04.prod.outlook.com (2603:10a6:800:b5::13) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4713.22; Fri, 19 Nov 2021 10:22:19 +0000 Received: from VI1PR04MB5600.eurprd04.prod.outlook.com ([fe80::8062:d7cb:ca45:1898]) by VI1PR04MB5600.eurprd04.prod.outlook.com ([fe80::8062:d7cb:ca45:1898%3]) with mapi id 15.20.4713.022; Fri, 19 Nov 2021 10:22:19 +0000 Received: from [10.156.60.236] (37.24.206.209) by AM6PR0202CA0058.eurprd02.prod.outlook.com (2603:10a6:20b:3a::35) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4713.19 via Frontend Transport; Fri, 19 Nov 2021 10:22:18 +0000 X-Outflank-Mailman: Message body and most headers restored to incoming version X-BeenThere: xen-devel@lists.xenproject.org List-Id: Xen developer discussion List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Errors-To: xen-devel-bounces@lists.xenproject.org Precedence: list Sender: "Xen-devel" X-Inumbo-ID: 94d9d9ca-4922-11ec-a9d2-d9f7a1cc8784 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.com; s=mimecast20200619; t=1637317341; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=Poc0wRx/mUi3cWDnszzqwQVxElnm2r9mLP8h58FAsVw=; b=F4i+wOnaUYE/o7juzWWY3omAp4NpvXnLgNxoQ6pWOVqe4ghTq/BsyshvILQOldv8qoXLut DKS+T2Kl1wvxNBEZmU0LpS67LWbBA1zwjiHOl0dHfxkZDPWaFQRerdNjcIBVZCI5tDBgy3 Irw8SjblW6btLPSLULmQOZ1l/WmCYAg= X-MC-Unique: AXYmrSAsPE6VtmYQLeNg8g-1 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=EUpd8WeQi5qeK1VJ/CD9bcv14ONyU7aU0qCno6b3zKzpYnsS7VerEDG47GbEBjFiKcqv9qjaqqNBxabZPiAlgDmvFzOmMXcsWRyKgSlhV0PtDoYWC/R5h6Pdgij2O5AyzMLX3k8xz41LX2+aaAwtw2Du5RTS0q5ehSmN7mxMFN/4vCliERfAeyy/lnfsGdVZPti5e+uox8cOBLaNR3cGVb+qKn/ya17HtZVzAT0SVHU971Ss1m6k01qUsRyDTgHlxlrY9XOQp8gIdVv3fT0gYs5bsJLjQUeFc1ynOl60vVHVYimOUmEASl9KOTrLq611H35AZroYLMQUvUUnReMA6A== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=Poc0wRx/mUi3cWDnszzqwQVxElnm2r9mLP8h58FAsVw=; b=AbhQdo828vE7c2lcOyuV5VoCvRqcc6BxTVpLOLKCRV/B9kYqi9Lzzvo2GDnFNPXxRQuILBubZrUVrStRGow/A2xTN/VhdqAdX58HPwps++UxIq9yWPWjHDNpEVx1sozXUpjxJ05TlCIvaNEWbWiAlEAd182wSsaLR4llSR4802P1yyXukA5+QMkvTmp2CJQc/gMHX0AfbOxnEGpabvkS10R3GBofmHb+4fYdgEJ/x8cugukPdvZGK6uqroWg4Q554gvAEZoUuc5OlJ2I3EyMXopq3r0TWzGsLKw+haqF8m6BLIqDw4XxnSf5lsKny7xYQ9xRk/eZfOagONYoB8cqpQ== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=suse.com; dmarc=pass action=none header.from=suse.com; dkim=pass header.d=suse.com; arc=none Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=suse.com; Message-ID: <389df3f4-5842-0f44-c8a4-42130f51959d@suse.com> Date: Fri, 19 Nov 2021 11:22:17 +0100 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:91.0) Gecko/20100101 Thunderbird/91.3.0 Subject: [PATCH 4/7] xz: avoid overlapping memcpy() with invalid input with in-place decompression Content-Language: en-US From: Jan Beulich To: "xen-devel@lists.xenproject.org" Cc: Andrew Cooper , George Dunlap , Ian Jackson , Julien Grall , Stefano Stabellini , Wei Liu References: In-Reply-To: Content-Transfer-Encoding: quoted-printable X-ClientProxiedBy: AM6PR0202CA0058.eurprd02.prod.outlook.com (2603:10a6:20b:3a::35) To VI1PR04MB5600.eurprd04.prod.outlook.com (2603:10a6:803:e7::16) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 793a95cd-4571-433f-1300-08d9ab46778b X-MS-TrafficTypeDiagnostic: VI1PR0402MB2704: X-Microsoft-Antispam-PRVS: X-MS-Oob-TLC-OOBClassifiers: OLM:9508; X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: FVZJZ5DaSMlNSR9yPTHe+OB+gbofu4kHpKQMBifucWnbtwH72y+4fGhSFPYuOS0p6/qJjI0Oh6mYP6MYPskU1O5QLgiF7+IicIiQzyClu9pzk8RNee4VhNCAGntPn+T2VinGEEn+0quXxz53J/IDhL+E5apiZDSZL1LNHmwt7kHI4lNmCZZKlr+iBABWPjtgpgnJ62CVF5AR8hrBO1WWSKuYOfF6C6/neyo9d3C9vKQjkWK6fGBOkCOTdPfA/AR6wTVEDtp4vRu17U2CJfRen2ipB+tqxxOO4qIz4g0uAedIl93DTr+GjrINxuqzlK5JRd/3vnLb7/uqHbDeQLno8MqdKVvHbLetjP2klcabkUN70dAoZUahFVM4MWgTcglRu0hbeQNzVmXRXPjOzZM1hCvyJcgRcG2HtN3Giq/Y/+ilNe4zg33kem1cA5Vcz7CEH38abpl6W3tNjjrxWnfaSVrAmkY4Ua2LxodT/cTvXEUUyFMns3TXZvYV4A7BPasYffX+NVSwtGC0S9f76ecaWfVXcc3PvusUPNo2Yzx+bFDNbysTtFYL+RKWLDIRSivlVu6C1vAOHwuA5yXUfsasfxErsHvoZtOu4wDREr4eK6kmA0OAq43HfMewhC8z7gMKz0HAKoLjxA3hqjdHj84gADHcEvAl8VAaSXCPexJl7FfccpU9kuSSSuGTLJMGd+n+6yq+uySQKxSVwPicCg3MmjW/kj3oXyy/DnTeBn/1vinSnYslX3VTQ82RbJSav6EOKf7wiCjkJvegQIr45A8cFEh/fYNwHeOAcr3Al4DWT7MPfQ3czU21fLllNcnNxvITxwbn5iNV9vK/LjeKdquRCA== X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:VI1PR04MB5600.eurprd04.prod.outlook.com;PTR:;CAT:NONE;SFS:(366004)(31696002)(38100700002)(36756003)(508600001)(2906002)(31686004)(8676002)(66476007)(66556008)(86362001)(956004)(2616005)(5660300002)(26005)(8936002)(6486002)(186003)(316002)(966005)(66946007)(54906003)(6916009)(16576012)(4326008)(45980500001)(43740500002);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?ZENlbzdSN2diWHZRdzREcUNCTForZnhBdENzL3A5M3h6c0ZHbmp3aG9rSk5j?= =?utf-8?B?dVA4OENxT0orc2JFczE3UVpJL3dCenpJcUdTQXVTdkdVeFNRRlM2VGwzNTlq?= =?utf-8?B?SEFhamkxNGUyeEpXRTd2T2FOdzNDeGtaM2VlT0pUbVM3anBmYXlzUVRBMVY3?= =?utf-8?B?QjNKNFluWDJTZmxpdnh2Mmhyc3BtTTQ4c1ZhVmYySTFPWmJzam1oOWxlRldK?= =?utf-8?B?YlhmVHdaZ0w3UlMzOUVMc0tiMHR4ZVZyYUhhUW95MTZKMWxlUCsvTkRhSS80?= =?utf-8?B?V0htQVUvVlFkUVVhVzIwMVdmeW5MTWNTNWNOQ0ZHUG5HWDJIWFBDV0lDbWZp?= =?utf-8?B?dkVKRnZIT0J4ZVJxVzBjWHVwVVZwSW0wejJldGNxQ1JiSjBvUk9XWnBYeFpk?= =?utf-8?B?b0QrZTVhNHZPSW9pMnF0UEpGYjl2anp3VTJxZmtYWUtxQjArUWJTWklBazZJ?= =?utf-8?B?OTFILzJVcm00Z0J3VzRhMG16ZjA4VmkyVnF3bDBUcmkvbnVJUjNLc2RwWDlR?= =?utf-8?B?NjY0M0VCTVAvLytuekpyZzBzU3p2Q0cycVY1SnE2R2hhS1h0a3N1YS9VenhM?= =?utf-8?B?ZTFJNzNkd3Uwd2ZKNzJiMFpPN0FGcjNvMUNHS2ZZb1I0aldheE51dzhnSmtp?= =?utf-8?B?d2J3WFRQN21ibzZDQ090RExBV0E4bU42UEZ6RlZKR0lXUkl3TFBZbTRQRE1V?= =?utf-8?B?MCtHWTdXaU5ySEg0bkdmbnljazM1S3JOYWFSVTg0MnlrSDc3MUhuVDNpdVdr?= =?utf-8?B?bzB5SFl0Z0ZjYnRRN3k2bDFzaURGKzhYQmhaSEpBZ0M0Y1ZqSUNLdW8wMVZM?= =?utf-8?B?R2dDOTVlSFRMWFJlWFFCdnpUcVhVQS9VNWd1OG1CWERiWnpnb2JyeEFjM1Iy?= =?utf-8?B?NTFwNHliTHl3cFAzRnRGbEdYM2U1cFN1S3E5dngvbkNlN0MreVhGdjEzYlBT?= =?utf-8?B?bW5FdVJsbHJhWnpidzV2d2pzbC9pK3MzZS9hUWVTcmZ2eFFiVVdqWHpOWkMw?= =?utf-8?B?dFd4Sm5aUlZvWjZtRlJOMlJScml5UU9kUDJQWWk0WjUwU254S2o5SjdhNnRk?= =?utf-8?B?QllRWmxVWWFQdzM4dlpiUkVuajFCVWhVUDdvNXBhYjdhZmFQSTFVd2dZbG9i?= =?utf-8?B?SktJd1l0VnErV1Vhalp1MTVETG9zTzM1SW9KQmpKemlxZWw1SGNRNjlmWXRO?= =?utf-8?B?QTdObE5TbC94Rm9OSUNlZzl0M2RPenltY3ZQRk9GRERuQTFRRWdMS0N4U3ZM?= =?utf-8?B?KzNtT0JnVnBCRlhIYTJOQm5STzE5dEZMeklFYmlWdzZoZi95OHVnSkViQjBn?= =?utf-8?B?aWc5U05FcXg5bjFlR0tScUhlUGVVUGhZRmh5anZxckhUOWcwOElJcHk4cHg5?= =?utf-8?B?UEIyeXd1QTgvZnAyOGVvVVR6a1RuZVVrWFh4VEtuRlRzbGFPQzVQNGRWckl3?= =?utf-8?B?d1ExdnJkTFhRTVQrSGpHNTNtL1o4dEJ1NWVEV29VZjBiZXMyMHYxcXBBQXdp?= =?utf-8?B?aDlzbkpFQUxRcmtONlNwOHBBTU84L0RETUFzaWZYMVBmN1JLdzZPdm4rWXJR?= =?utf-8?B?WGFHbG96a3o0R1pFQUhVcFhsN2FtNUdsUUJld0lzZ3dCS3N3cGxqK3JxclVp?= =?utf-8?B?QnJKaCtmRm5FbmpLTy95ZzhpNFVuV1VOR0k0YkUvY254MEhLMmFxMVhXandQ?= =?utf-8?B?VW5RTXB5Z0JUY0hmR2NHOEhYRXBLODVQRnpGZkdmcmVKdHdaK2JPZkxnOExw?= =?utf-8?B?ajRiYnFVbWZXZ1ZWZ1JGZnBramlVK0hvY1k4cmdxQXI1SlI1cENqdTBJZTl0?= =?utf-8?B?amhSN0pEWWxkcDJaekVOMjVoZGQxUVhnN1h1bHEyN3MxN1BLNFBJd3lmcnF4?= =?utf-8?B?Sjl3cjdHd2VVT29YZGlWWjNDK3A1YmtLeDBNTFdFNk1ZNEg2c3NKYi9qOGxu?= =?utf-8?B?M2pQQWQ3NENwcUVsNysycldSTXBzZldvcWJCNlZQbEhSOWYwQnlSSi94emFO?= =?utf-8?B?WS82TFNMaU94OE01NEh1Yk85RDVTS3llcE5DdWMwQTdjNUg0cm81S3dxU3gr?= =?utf-8?B?VHMzTHduMVpGQURmTitIYjlNZlViYmlUWjdkd1NIbFBiL0tKdlhiSVp3NGFS?= =?utf-8?B?ekdCVERLVzYvWU9qNWk5TnpDclVoR0JTbkpBbGtaQzM4NzlrZVNtdE5jWmlp?= =?utf-8?Q?U8i0iChKSnhA6P0JVkZ5D3Y=3D?= X-OriginatorOrg: suse.com X-MS-Exchange-CrossTenant-Network-Message-Id: 793a95cd-4571-433f-1300-08d9ab46778b X-MS-Exchange-CrossTenant-AuthSource: VI1PR04MB5600.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 19 Nov 2021 10:22:19.2528 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: f7a17af6-1c5c-4a36-aa8b-f5be247aa4ba X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: dH25xuGdz1U/svWoeddzKVBIS/sm3m3jNLulq4nz5gGIXo8rhYf+C5GtDpCA8pRz8PXZqwxYsT0tVYPcVMlsYg== X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR0402MB2704 X-ZohoMail-DKIM: pass (identity @suse.com) X-ZM-MESSAGEID: 1637317359168100003 Content-Type: text/plain; charset="utf-8" From: Lasse Collin With valid files, the safety margin described in lib/decompress_unxz.c ensures that these buffers cannot overlap. But if the uncompressed size of the input is larger than the caller thought, which is possible when the input file is invalid/corrupt, the buffers can overlap. Obviously the result will then be garbage (and usually the decoder will return an error too) but no other harm will happen when such an over-run occurs. This change only affects uncompressed LZMA2 chunks and so this should have no effect on performance. Link: https://lore.kernel.org/r/20211010213145.17462-2-xiang@kernel.org Signed-off-by: Lasse Collin [Linux commit: 83d3c4f22a36d005b55f44628f46cc0d319a75e8] Signed-off-by: Jan Beulich Reviewed-by: Luca Fancellu --- a/xen/common/unxz.c +++ b/xen/common/unxz.c @@ -127,7 +127,7 @@ * memeq and memzero are not used much and any remotely sane implementation * is fast enough. memcpy/memmove speed matters in multi-call mode, but * the kernel image is decompressed in single-call mode, in which only - * memcpy speed can matter and only if there is a lot of uncompressible da= ta + * memmove speed can matter and only if there is a lot of uncompressible d= ata * (LZMA2 stores uncompressible chunks in uncompressed form). Thus, the * functions below should just be kept small; it's probably not worth * optimizing for speed. --- a/xen/common/xz/dec_lzma2.c +++ b/xen/common/xz/dec_lzma2.c @@ -387,7 +387,14 @@ static void __init dict_uncompressed(str =20 *left -=3D copy_size; =20 - memcpy(dict->buf + dict->pos, b->in + b->in_pos, copy_size); + /* + * If doing in-place decompression in single-call mode and the + * uncompressed size of the file is larger than the caller + * thought (i.e. it is invalid input!), the buffers below may + * overlap and cause undefined behavior with memcpy(). + * With valid inputs memcpy() would be fine here. + */ + memmove(dict->buf + dict->pos, b->in + b->in_pos, copy_size); dict->pos +=3D copy_size; =20 if (dict->full < dict->pos) @@ -397,7 +404,11 @@ static void __init dict_uncompressed(str if (dict->pos =3D=3D dict->end) dict->pos =3D 0; =20 - memcpy(b->out + b->out_pos, b->in + b->in_pos, + /* + * Like above but for multi-call mode: use memmove() + * to avoid undefined behavior with invalid input. + */ + memmove(b->out + b->out_pos, b->in + b->in_pos, copy_size); } =20 @@ -421,6 +432,12 @@ static uint32_t __init dict_flush(struct if (dict->pos =3D=3D dict->end) dict->pos =3D 0; =20 + /* + * These buffers cannot overlap even if doing in-place + * decompression because in multi-call mode dict->buf + * has been allocated by us in this file; it's not + * provided by the caller like in single-call mode. + */ memcpy(b->out + b->out_pos, dict->buf + dict->start, copy_size); } From nobody Fri Mar 29 08:33:21 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) client-ip=192.237.175.120; envelope-from=xen-devel-bounces@lists.xenproject.org; helo=lists.xenproject.org; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) smtp.mailfrom=xen-devel-bounces@lists.xenproject.org; arc=pass (i=1 dmarc=pass fromdomain=suse.com); dmarc=pass(p=quarantine dis=none) header.from=suse.com ARC-Seal: i=2; a=rsa-sha256; t=1637317387; cv=pass; d=zohomail.com; s=zohoarc; b=dMAQe/BZzsop0an8+Q2lRmkM9T2Z8BvDUM8bi58Q1PjvQNtK44DYYGQ+ttxc25gHwXKnXjM9ASef6zuj1u2Hykd8Z6f3pv4jy5IOs7PRvY0SrIKKAX34n2gIUe9TnPizPtNufJsKhY7pQDBZLoeWqjPpn/iRCectCSNfFGzEaPg= ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1637317387; h=Content-Type:Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:To; bh=+YZ/RYh6HZJE7d4sF/mnmyPw4f+smxi+AYdlGT8UcGM=; b=NIpGfMdjnHJd64lB5hvGNVH8Hlu+NApfj5ZkT/i8zjsMB7zRnOFBZoqXPV3iruvoc384VY/4hjwZzHwnvIs5pB4sjaX8zxJPh1mlrrY+dZi3f1lmOL9HNx/F2+dqABpv58LDCVJZip7Z47Z1ktceYxlWXwbMcxOSvwbKspluR8E= ARC-Authentication-Results: i=2; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) smtp.mailfrom=xen-devel-bounces@lists.xenproject.org; arc=pass (i=1 dmarc=pass fromdomain=suse.com); dmarc=pass header.from= (p=quarantine dis=none) Return-Path: Received: from lists.xenproject.org (lists.xenproject.org [192.237.175.120]) by mx.zohomail.com with SMTPS id 1637317387489340.23941398570037; Fri, 19 Nov 2021 02:23:07 -0800 (PST) Received: from list by lists.xenproject.org with outflank-mailman.227736.394003 (Exim 4.92) (envelope-from ) id 1mo12d-0004GZ-9L; Fri, 19 Nov 2021 10:22:47 +0000 Received: by outflank-mailman (output) from mailman id 227736.394003; Fri, 19 Nov 2021 10:22:47 +0000 Received: from localhost ([127.0.0.1] helo=lists.xenproject.org) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1mo12d-0004GS-5F; Fri, 19 Nov 2021 10:22:47 +0000 Received: by outflank-mailman (input) for mailman id 227736; Fri, 19 Nov 2021 10:22:46 +0000 Received: from se1-gles-flk1-in.inumbo.com ([94.247.172.50] helo=se1-gles-flk1.inumbo.com) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1mo12c-0003FS-5V for xen-devel@lists.xenproject.org; Fri, 19 Nov 2021 10:22:46 +0000 Received: from de-smtp-delivery-102.mimecast.com (de-smtp-delivery-102.mimecast.com [194.104.111.102]) by se1-gles-flk1.inumbo.com (Halon) with ESMTPS id a2e55cda-4922-11ec-a9d2-d9f7a1cc8784; Fri, 19 Nov 2021 11:22:45 +0100 (CET) Received: from EUR05-AM6-obe.outbound.protection.outlook.com (mail-am6eur05lp2109.outbound.protection.outlook.com [104.47.18.109]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id de-mta-18-WQAgmwXzNoOlNJ4ZFus9Iw-1; Fri, 19 Nov 2021 11:22:44 +0100 Received: from VI1PR04MB5600.eurprd04.prod.outlook.com (2603:10a6:803:e7::16) by VI1PR04MB5168.eurprd04.prod.outlook.com (2603:10a6:803:56::33) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4713.22; Fri, 19 Nov 2021 10:22:42 +0000 Received: from VI1PR04MB5600.eurprd04.prod.outlook.com ([fe80::8062:d7cb:ca45:1898]) by VI1PR04MB5600.eurprd04.prod.outlook.com ([fe80::8062:d7cb:ca45:1898%3]) with mapi id 15.20.4713.022; Fri, 19 Nov 2021 10:22:42 +0000 Received: from [10.156.60.236] (37.24.206.209) by AS9PR04CA0059.eurprd04.prod.outlook.com (2603:10a6:20b:46a::9) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4713.19 via Frontend Transport; Fri, 19 Nov 2021 10:22:41 +0000 X-Outflank-Mailman: Message body and most headers restored to incoming version X-BeenThere: xen-devel@lists.xenproject.org List-Id: Xen developer discussion List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Errors-To: xen-devel-bounces@lists.xenproject.org Precedence: list Sender: "Xen-devel" X-Inumbo-ID: a2e55cda-4922-11ec-a9d2-d9f7a1cc8784 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.com; s=mimecast20200619; t=1637317365; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=+YZ/RYh6HZJE7d4sF/mnmyPw4f+smxi+AYdlGT8UcGM=; b=OrxysEkAe9gTIlmpR8eJCEWQ1yF5u9++tp0NLlW3eYtTahxU/XHmk9NHEhec8twuRn8BMO D5r4j634AIOC7uTC+28zd9ZFsiLsPC5VfuNr+BKEvf7wwOgXox/Cwfp+iUk0uRnMyTX7D7 OxsKJ1I5SizbNDhE79wAUSbatqggFtk= X-MC-Unique: WQAgmwXzNoOlNJ4ZFus9Iw-1 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=ZrEhwEPkLd/189ebi1bAm5dF3wwonPRUYdJDPN/rZDFQ4gAgCp0tbuHBpCm1I45OndffIpw7waLZVxwezzf3OaIg7vl9279Y4UOcWZfiDB36Zs2csRUTbviEkhMw/D4jNZW7NMI1N65mcbRY6pxwb5aIDPDG/q9KZWZxReTe4YtCFpykkwXlWIJdHPYzfZATAjnpgrJo2pEAlK3C8q23I8W5PoEERN8MEJ/BJxD133zjzrdgXtK0tNBIy2fnRtpHkpC+YNfBipWaoMW2z7ly8yyQspShMuBUi+FESIBzKaMWucP1YJCGD7ivT7s5n5Z2Ost6U/nclrTx3NIOF4+O0Q== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=+YZ/RYh6HZJE7d4sF/mnmyPw4f+smxi+AYdlGT8UcGM=; b=mKi5bKOMTvmAs3jypacP0i9/VX0ipRfDwak1R59ix1Kky9fNFi7YFWxEYv1VZ3qvz7Ib3eqZ22nNlis/ImgxfG+0/XbKjC9jwN1V2nZW32h+DyMtrEG3ajxnqvhM2wdHr0vfxCFFya98XqeeS8jWsYKWsuz07aF4Om+xMuIqcL7wkI7+yf1j1EQM8sxV0tQ3/AaZS0u8+Zi1vU+jsRgiZHnx7Dw1MocFvvaU2buhlr/F5k0VccY9809TNF6kDocdeLvCUuL1AAWaXd1aQSRpaG4O6Hhpk4KXV98OMOEjP1iMwxasbdZQU8YFG3EbcT4SmkqZj4Rmrf7kXmt4y/x96A== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=suse.com; dmarc=pass action=none header.from=suse.com; dkim=pass header.d=suse.com; arc=none Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=suse.com; Message-ID: <3b6557c0-08ce-b4ae-87e5-ff550a07ec9f@suse.com> Date: Fri, 19 Nov 2021 11:22:39 +0100 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:91.0) Gecko/20100101 Thunderbird/91.3.0 Subject: [PATCH 5/7] xz: fix spelling in comments Content-Language: en-US From: Jan Beulich To: "xen-devel@lists.xenproject.org" Cc: Andrew Cooper , George Dunlap , Ian Jackson , Julien Grall , Stefano Stabellini , Wei Liu References: In-Reply-To: Content-Transfer-Encoding: quoted-printable X-ClientProxiedBy: AS9PR04CA0059.eurprd04.prod.outlook.com (2603:10a6:20b:46a::9) To VI1PR04MB5600.eurprd04.prod.outlook.com (2603:10a6:803:e7::16) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 28b23aa6-b3ab-4535-9a28-08d9ab468525 X-MS-TrafficTypeDiagnostic: VI1PR04MB5168: X-Microsoft-Antispam-PRVS: X-MS-Oob-TLC-OOBClassifiers: OLM:10000; X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: jNiJC2DIds0oOOyRQOhc77xEXJHEP0YrMBD3bfDdi+03CmnXReqvRfFPNyR83B+3NubG+W2QC0zewEzIqug5sa0d+kBTL9TMP0iJK7hYcms+dfFmcuOrKI7GRYjYgadKlZ6wdYD9z36ybWXVfT3dvP9sFqrjQgS3pBYt0oyr5WaEcnIQPf+hiYdwhdo7i4rFMN+TMMIzSBuVkcDScjGr5+oTcIKy/gv33DmrjtNq8JYabsBzQ81el210C6zQH612//2AMMPJHYOndix1+YwAe97NlzCDuqd8M6NpwuDB3pHne9zVkG2tifFbK5mdfs3azCqFr2JMY5G2mNq8p/0C5YI4iQyfvzG7UScpw8RSqA/Kvda820hl6pa6IxbNHdlfywPObJkALx50j0uGQMjHBBkfVZjR10kmR8tTpQbyN+ax1H3UoqerXlPovB7Cf+e952yYGfwjunHoDzhnPq/QfaL108/5LHCOARZAqe1+pswPjW54UxWPJF1zK2RuyFpQSh1XZf5AIQDXYyr74AokMWwWHaLt1d41bP6qh+/b6ePRh0LeP1ANQy9MV3l5o42GkgSrIH9qTpkkQ31a2FwGTTAUZugr85Ev/lqmV8FKvC3ZeU9r9naU6nkgyQRYW1bXamOMYLZET9GQ34o2CepNSzQDxqWqqQyBWfnAxDmUETqRWnSp1YzaCvO8XEyn/kyRs9eBbaAQLjJjYxEOykAL5HMjiL/pwByQOT/Tqca2C/eQEUtZcEr15Tze7WBTDPquE3M87p5sTVmq432ky0gaN/VzPOsd49Q1Fgeuuvl3Dl42Zyj8URZSjCEZMlRXyAZy X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:VI1PR04MB5600.eurprd04.prod.outlook.com;PTR:;CAT:NONE;SFS:(366004)(6916009)(38100700002)(66946007)(316002)(6486002)(16576012)(8676002)(966005)(86362001)(31696002)(2906002)(186003)(956004)(5660300002)(2616005)(4326008)(66556008)(54906003)(8936002)(26005)(66476007)(508600001)(31686004)(36756003)(43740500002)(45980500001);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?dFU0Q1E3elJDZGE3TktrbCs2SFRSdkhZbmtMRWhOSW0zRWxoNEdaUUJGMHNx?= =?utf-8?B?TlRxWmwycCs1R0JPOUZTbStIdkg0L1VNeEdhc2JGaDM4MGY1Ni9yalN3dTZx?= =?utf-8?B?U3EzbnVUeCtRMzNpajlPOVNIYUd4U08wcU80cjBJeTUzbEZCYTlGZ2tVQXhU?= =?utf-8?B?OENWZ05JUEhVbUR2azRoeHZKc3FWSTIwT1p2WTV2OXhnNE1acFpwQWI5OWR2?= =?utf-8?B?azBUdkozWFhHSUIwK2E4SURDYkJsdUk1clNXUHZTOEg5WitnS2FsbVh2MHhU?= =?utf-8?B?V2NlVXZxNms3SWxhdThneHZneUsxK2xoQ1ZvVUFlcjlkRzI1QmlaQVR5MUtr?= =?utf-8?B?cWVoUmZTYXorUDhYcWZXbUc2RW5KWjZPMjE4YTlUd2pYTVgxbXZqRFdlOFZE?= =?utf-8?B?WGpycG1yNFhBTDV2aDRDc0h4Y0tmWG93eXcrTDVkWGNnT2d2RCtWQnowZ3BX?= =?utf-8?B?ZngxN1ZzeUxxellRdStuL1NNc1RadU16d284aXVWY2dUZ0UwUW5wMmFIL3NE?= =?utf-8?B?RWFpTGYzWFNlNUJXYzlKWmNjcWg5ZUtqR3ZZOW43SENXSXUzeWhhRGJLL2xV?= =?utf-8?B?U0dZdjI1LytpVTFWNzB0RnJGdGJSVmhSbEpNM3FzRFlLelUyNldBK2s3REFk?= =?utf-8?B?dGV6WGZldmRpWHRocnB0NkhRVG42ejdXZkxOWExWZDBGTS9RNVdLUXliRnVs?= =?utf-8?B?OTRkcGozRVZ3TkJSUy8vSkpTc1hSZzBzc1g4ZUcrRkhSVDJsTEVqSXFoYWFk?= =?utf-8?B?LzdzcmFuUUltOHM0L0d6ZWY2REFKVUxuQWtJaWl3cVBldm9WSG5Eb21KWnpa?= =?utf-8?B?QUx3ZUV4WTB5SWdBa1pvbndKT3RDNEtSb1I1VkxERTBPRG1QU09JeTU3ZDJV?= =?utf-8?B?azNSTTRBV3ZLQkFNV0FLK2VCWDh1NG1VTk1KclY5ZWw3UHlyWi9yNUMzTXRX?= =?utf-8?B?blZlN3Z0SVVrS3F4U0kxdm94ZUhtcllKVVVwWWRtRUVodnJ1Um82MXNyVzBF?= =?utf-8?B?VUIzdWVDU0I5ci9ZdnhhR05LTnd0UnRMaGNXU0tuTzFvYzVoWVhxcEFZS2hl?= =?utf-8?B?L0tINENRRWplUjNoTGZJS3JzUTVBdDYxWVk0am1ndzNwbmV0R3B3TDRFMTdL?= =?utf-8?B?WXRZREN4VFd2SWlDSkk2bmREcHF5K3NVNXZoK0pJdzdCZWNEVVl3MGxqYzl5?= =?utf-8?B?Slk1UEwwY3AzUjAzWkwyenkrQUlaMmljdlB2dkN1YXRxWThpV25UVTZBNzdq?= =?utf-8?B?SFFERDBSYlM2U2tTK2lkbU9yaU9LalFJb1pjVWJjRC9iR0U2WFVZMFJIam1B?= =?utf-8?B?NkFPMVlYRzJHbDVnSFE4UW1sN0hwQmN1Q0dKemNheUJjRW1IaXIrNEJhWDhx?= =?utf-8?B?Q3hFQTZDYWM3dUVFWWxrSGNydGlIQnVCSklKVWU3S1h1L20rbzRRclRIemZj?= =?utf-8?B?dkZaZ0tQeVpCa3dJUjh1SlMyR29hNWU1Y2pKK3M0Q0pLaUgxU3MvZGhJMS83?= =?utf-8?B?WTFIQTlWWG8xVm1GV0xrRWdSaWM0UVErVGdmSVQ3RTlYVmY0QWZORU9zaHR6?= =?utf-8?B?bzYxV2hqNGVTaGgxVzUxTnFvMWF3SXRFd1hZczdoL0pwU0ZCRFhlVXVLRUVj?= =?utf-8?B?dVpTMTNKN2svUUd6V3ByOXRhVnFYSmova2U1NnYxWDNVRDU2aXMzcFkrMVlP?= =?utf-8?B?MnA5aDQ4S2NBSyt1bUZkUXJMQXFjYk9KT2ZrbGhPVHVPRFJ4NUswMnhtSk5p?= =?utf-8?B?V3REMUJLZkYvWUFuS1djV2Q0cE1yaEVTeFdBQVVPeTl0TVdHcVZWaUQrcENN?= =?utf-8?B?am50T1B1MmhkSXFiaWNhYjRSbFpMVTRvc0RHdkJ0ck8yZEkwa2lQSU92VmVV?= =?utf-8?B?dVhpWEZoQWVJMUViWFFnbk5PTFNHSFQ5N3h0T0NlR2Z3ZUxZRmp6Tlc3dGw5?= =?utf-8?B?QTlWUzBMQmFKc1hac1FhdGw4VjY3UmJpejcxbHZkdFdscDEzVlFodkdQSTJO?= =?utf-8?B?T3RVZ2RidDdIelIvaHZTNWp0RlhzWnV5MHd2Sk5DOE50Wk9xRTZVMEhtYVc0?= =?utf-8?B?bUxUMUF0QWxCK1Qzd1l6ODJiVDNGZkdFVkhlY1ZPYlVnMHNLLzR4MDRwREZu?= =?utf-8?B?SWNUZGpwcmxzMThnbGxDSy9TdHNEalg0R1RxU0hGQzFyVW9uUEREMWhoYjF4?= =?utf-8?Q?PHH+z3ron7rmLiQyptYS3C0=3D?= X-OriginatorOrg: suse.com X-MS-Exchange-CrossTenant-Network-Message-Id: 28b23aa6-b3ab-4535-9a28-08d9ab468525 X-MS-Exchange-CrossTenant-AuthSource: VI1PR04MB5600.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 19 Nov 2021 10:22:42.3118 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: f7a17af6-1c5c-4a36-aa8b-f5be247aa4ba X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: BIlacKb+4217UHvK2OWAe3bfYepRy1mrgDg8OPr6DKgeB7/EyUZfwgeQPJIzD79KjcWmSTPzQ1oFmrT587FYhw== X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR04MB5168 X-ZohoMail-DKIM: pass (identity @suse.com) X-ZM-MESSAGEID: 1637317388862100001 Content-Type: text/plain; charset="utf-8" From: Lasse Collin uncompressible -> incompressible non-splitted -> non-split Link: https://lore.kernel.org/r/20211010213145.17462-6-xiang@kernel.org Signed-off-by: Lasse Collin [Linux commit: 0a434e0a2c9f4395e4560aac22677ef25ab4afd9] Signed-off-by: Jan Beulich Reviewed-by: Luca Fancellu --- a/xen/common/unxz.c +++ b/xen/common/unxz.c @@ -20,8 +20,8 @@ * * The worst case for in-place decompression is that the beginning of * the file is compressed extremely well, and the rest of the file is - * uncompressible. Thus, we must look for worst-case expansion when the - * compressor is encoding uncompressible data. + * incompressible. Thus, we must look for worst-case expansion when the + * compressor is encoding incompressible data. * * The structure of the .xz file in case of a compressed kernel is as foll= ows. * Sizes (as bytes) of the fields are in parenthesis. @@ -58,7 +58,7 @@ * uncompressed size of the payload is in practice never less than the * payload size itself. The LZMA2 format would allow uncompressed size * to be less than the payload size, but no sane compressor creates such - * files. LZMA2 supports storing uncompressible data in uncompressed form, + * files. LZMA2 supports storing incompressible data in uncompressed form, * so there's never a need to create payloads whose uncompressed size is * smaller than the compressed size. * @@ -127,8 +127,8 @@ * memeq and memzero are not used much and any remotely sane implementation * is fast enough. memcpy/memmove speed matters in multi-call mode, but * the kernel image is decompressed in single-call mode, in which only - * memmove speed can matter and only if there is a lot of uncompressible d= ata - * (LZMA2 stores uncompressible chunks in uncompressed form). Thus, the + * memmove speed can matter and only if there is a lot of incompressible d= ata + * (LZMA2 stores incompressible chunks in uncompressed form). Thus, the * functions below should just be kept small; it's probably not worth * optimizing for speed. */ --- a/xen/common/xz/dec_lzma2.c +++ b/xen/common/xz/dec_lzma2.c @@ -505,7 +505,7 @@ static always_inline void rc_normalize(s * functions so that the compiler is supposed to be able to more easily av= oid * an extra branch. In this particular version of the LZMA decoder, this * doesn't seem to be a good idea (tested with GCC 3.3.6, 3.4.6, and 4.3.3 - * on x86). Using a non-splitted version results in nicer looking code too. + * on x86). Using a non-split version results in nicer looking code too. * * NOTE: This must return an int. Do not make it return a bool or the speed * of the code generated by GCC 3.x decreases 10-15 %. (GCC 4.3 doesn't ca= re, From nobody Fri Mar 29 08:33:21 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) client-ip=192.237.175.120; envelope-from=xen-devel-bounces@lists.xenproject.org; helo=lists.xenproject.org; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) smtp.mailfrom=xen-devel-bounces@lists.xenproject.org; arc=pass (i=1 dmarc=pass fromdomain=suse.com); dmarc=pass(p=quarantine dis=none) header.from=suse.com ARC-Seal: i=2; a=rsa-sha256; t=1637317400; cv=pass; d=zohomail.com; s=zohoarc; b=YB+S5Lq2+ecXivc/VIdfkBLnleHvBbtUZFp8btOutNrmbDSmIokZEx8lhTTeuhKjW45YhVqeVZ+Ts1YECS5NBhbY5pA5S5YWSWobFoVsiBe+l7g+W4c2zGSQwKHfLUdxaLEkES8X7DAmqMErUZencQtnmOJdNNF4SjG+7n8dUsU= ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1637317400; h=Content-Type:Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:To; bh=SFdBl4oly+v5EDzv1v3dC+TsVpDheb1hphiY7sT/fCc=; b=Z0rhujz1Z0E/8E5hZdQXXvi32ZvHF0XOZM5YSZqqZKviRABHJDU8xZSF5wyDXi7TPAAXzyEFOxC6KLMlzh5X+H4zM5mFHIsI5pJbLRcK/RX+krVMH/aZDvZqvTgDpRwchtRO8EgE54z2m1T0lmajtSN2/59usBDEzbw1PD11O+M= ARC-Authentication-Results: i=2; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) smtp.mailfrom=xen-devel-bounces@lists.xenproject.org; arc=pass (i=1 dmarc=pass fromdomain=suse.com); dmarc=pass header.from= (p=quarantine dis=none) Return-Path: Received: from lists.xenproject.org (lists.xenproject.org [192.237.175.120]) by mx.zohomail.com with SMTPS id 1637317400527336.32281978128344; Fri, 19 Nov 2021 02:23:20 -0800 (PST) Received: from list by lists.xenproject.org with outflank-mailman.227741.394014 (Exim 4.92) (envelope-from ) id 1mo12x-0004p7-I3; Fri, 19 Nov 2021 10:23:07 +0000 Received: by outflank-mailman (output) from mailman id 227741.394014; Fri, 19 Nov 2021 10:23:07 +0000 Received: from localhost ([127.0.0.1] helo=lists.xenproject.org) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1mo12x-0004p0-EN; Fri, 19 Nov 2021 10:23:07 +0000 Received: by outflank-mailman (input) for mailman id 227741; Fri, 19 Nov 2021 10:23:06 +0000 Received: from se1-gles-sth1-in.inumbo.com ([159.253.27.254] helo=se1-gles-sth1.inumbo.com) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1mo12w-0004k5-6H for xen-devel@lists.xenproject.org; Fri, 19 Nov 2021 10:23:06 +0000 Received: from de-smtp-delivery-102.mimecast.com (de-smtp-delivery-102.mimecast.com [194.104.109.102]) by se1-gles-sth1.inumbo.com (Halon) with ESMTPS id aec93a50-4922-11ec-9787-a32c541c8605; Fri, 19 Nov 2021 11:23:05 +0100 (CET) Received: from EUR03-AM5-obe.outbound.protection.outlook.com (mail-am5eur03lp2055.outbound.protection.outlook.com [104.47.8.55]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id de-mta-37-iWGrbT8jPouzRdHVT6wxsw-2; Fri, 19 Nov 2021 11:23:03 +0100 Received: from VI1PR04MB5600.eurprd04.prod.outlook.com (2603:10a6:803:e7::16) by VI1PR04MB5168.eurprd04.prod.outlook.com (2603:10a6:803:56::33) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4713.22; Fri, 19 Nov 2021 10:23:03 +0000 Received: from VI1PR04MB5600.eurprd04.prod.outlook.com ([fe80::8062:d7cb:ca45:1898]) by VI1PR04MB5600.eurprd04.prod.outlook.com ([fe80::8062:d7cb:ca45:1898%3]) with mapi id 15.20.4713.022; Fri, 19 Nov 2021 10:23:03 +0000 Received: from [10.156.60.236] (37.24.206.209) by AS8P251CA0017.EURP251.PROD.OUTLOOK.COM (2603:10a6:20b:2f2::30) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4713.19 via Frontend Transport; Fri, 19 Nov 2021 10:23:02 +0000 X-Outflank-Mailman: Message body and most headers restored to incoming version X-BeenThere: xen-devel@lists.xenproject.org List-Id: Xen developer discussion List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Errors-To: xen-devel-bounces@lists.xenproject.org Precedence: list Sender: "Xen-devel" X-Inumbo-ID: aec93a50-4922-11ec-9787-a32c541c8605 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.com; s=mimecast20200619; t=1637317385; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=SFdBl4oly+v5EDzv1v3dC+TsVpDheb1hphiY7sT/fCc=; b=hXXqJR9cXAgsoMrY81CfjfTLdBbACqBSKOqY+JVarD4n/SxsaIBedd0eN+H+7tAzdL1/sF UFwRHm/6xalBIrEYEdGuQoCnXH3Ukv+QueD6OE22RVagXeR5uvrBCCYv0DU/6qLrGo7O2a ARpEw2KUgFa9ul45nl8ZVDz4Zd/IXK0= X-MC-Unique: iWGrbT8jPouzRdHVT6wxsw-2 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=dNemgzmkMLSJ/oACIcUgMxygPlisTFMU7uqhl2l3UArf+sfT6JwQaSKkL/82ThjkcTQpNteLC0+6jVn31IVTzF+X/7hPClkYwkUX+Q3zDJqdi0Bfb9BthWftILihR8O8bApPYxbvHadr0v1lberaD0Rh5mzNshCF97EqffpQvr5927a5AmyF0OnZ6srGUmieZa4wvk1NGqWNuQXAjyRrsiNMxKAduPddFlDNS5vQBbDD1zNUFAKaNBixo6OqF/Hx3S4xu9oApvv8ZNpAD35zUgabcf3nohyBC9Dg0ojwdaBsbsFQ6UEI9Io+b/2szFlisF+1aLSuXei8WnYn92hmAw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=SFdBl4oly+v5EDzv1v3dC+TsVpDheb1hphiY7sT/fCc=; b=mSL+a6A11WVESi64NgWk9HhsWhscKAoq2jIYgAmxGpR+hn2FF+faD31rWps0WStLSINnmnQ6P57G79xZSd/5GMEBB4K581LRIku8NfhtyQhG4OFVovRK5bQuWNWXpRc7ygCNWxArlVDm+xl++Ic5O3OT0znYOZ8ZgsmQ9zIE8AJ7lqLopUOLyAn8tUStXh1s94iPMur3BwSLJmb/pu07XgG2Q74ROM1Di7w4jDB+YHWsoU6RJrJkxB25naYSdKjsInt7on+JrEl8gH3b5vH3r9PayyisYIDAxOYErmRuqV1xgFT6g4YDAojMOGzs4w34mj3G2BZKphNgAzCDMHlkVw== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=suse.com; dmarc=pass action=none header.from=suse.com; dkim=pass header.d=suse.com; arc=none Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=suse.com; Message-ID: Date: Fri, 19 Nov 2021 11:23:02 +0100 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:91.0) Gecko/20100101 Thunderbird/91.3.0 Subject: [PATCH 6/7] xz: move s->lzma.len = 0 initialization to lzma_reset() Content-Language: en-US From: Jan Beulich To: "xen-devel@lists.xenproject.org" Cc: Andrew Cooper , George Dunlap , Ian Jackson , Julien Grall , Stefano Stabellini , Wei Liu References: In-Reply-To: Content-Transfer-Encoding: quoted-printable X-ClientProxiedBy: AS8P251CA0017.EURP251.PROD.OUTLOOK.COM (2603:10a6:20b:2f2::30) To VI1PR04MB5600.eurprd04.prod.outlook.com (2603:10a6:803:e7::16) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 47676349-3ed7-4dd0-2313-08d9ab4691a8 X-MS-TrafficTypeDiagnostic: VI1PR04MB5168: X-Microsoft-Antispam-PRVS: X-MS-Oob-TLC-OOBClassifiers: OLM:113; X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: o6Ve08ZqwddAUZnt8LUF6/sDPpgDmdVgiI1iiGI9kh6WJPEgm6JDFxjBNESYevFRCKxd+orWtLs9WawITiilp0AVa3tVftd07QPj6C+q5AkNmfU56MPW1Vyn39In+kapCdk+Vrz3wJYoEYkqmwg9Ule4iys6QCDgBLalNuis/Dpkqgx0jYYAEmmCHK6/7q30Q2XbTO7mkOWpFn2YrhEIZ4jtzzM39Cj17Pz4dIJBXD2ruYCFCwZYH8yNxNBQX3+y8pVQtg2y2LkGUp8SSKw1Qqw/B9XFnUsq4QgkXreBdKAzsHyigXYzRYjy//mvDK2GV+vuDayQk+ZRDBiytFIIAkFdt3mjoCyJPwVxWM3JeteH9EEFKCj5kmpZIt952UCnZYv9T1KrBE4zBkHSVU6YCJX/QAwvMHFR83zYGcidvENkYU2bwrxIWiGq7riWwGORae6cJc3g7VzdhcNiqsn2x0i4OUOyunfR4mtv4+FF/wdzK7J8bIDX057PgItIYH/SHX6apZX4vsJToof/qHQwLoInurH+8b6ITVMblBcB1JlVyUU82OVma8mxDfpDBZ1GAFk6qeHNm/5t7MHlariLO7I2Z6vtCGe9X9jPkbK+s3630ZD5MSkcDkpjPe+SGr9DudNeGArk46khMsWxHT9TshdFG+xJbpIqFm66KawkNOCjqK3m1BZrJmWcOF3mqKlZpi7Is2REDk/e5vI0PMVMpG2Ts2CA1T+kvnRe061LhvUfoS/OCF1nZX+HSqt5EkdTGlbfQfC9oNn3owxQv4P7TwBQGCVyACnHmXubC3OqvdGo83es1yhz6TksaqpXiw7a X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:VI1PR04MB5600.eurprd04.prod.outlook.com;PTR:;CAT:NONE;SFS:(366004)(4744005)(6916009)(38100700002)(66946007)(316002)(6486002)(16576012)(8676002)(966005)(86362001)(31696002)(2906002)(186003)(956004)(5660300002)(2616005)(4326008)(66556008)(54906003)(8936002)(26005)(66476007)(508600001)(31686004)(36756003)(43740500002)(45980500001);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?RlducmJaaXJSZWZVa21URVFUTjVqY2ozZXQ1RmVpTVRTQ3o5ZkpRSGZMRURu?= =?utf-8?B?U1htTHlSek00cTE2emRQREllTEp5cXBGdVFGVGV1UmNQL0dINTNyZGp6QVhq?= =?utf-8?B?ejBzdGo1TG51cmQ2NERrTjBuQ1RxTURvNVdiTGNaOThUeEhRYUJrSnBlMW1q?= =?utf-8?B?SVNIYnpNdCtvbHVDaGEyK0QxQ3ZPRjRsZndNYVFSYWtzRVFaVjIvdzY0YUwv?= =?utf-8?B?d2V4SjhFcHZYa0RRVjhPZGIwWWRmL1VCZXZCSXUxOHAwRG9xNmR6eThLZkJj?= =?utf-8?B?UnA3ZmJQMlBWSDFYSW5neWFpcXRnd3U1Y2dUQUJQZkRQZkVyVXBxMy9pSDVU?= =?utf-8?B?VFFYYnBSQWw3QlQ2dWI5MEp6dytCaUx1NHR4OU5UREY3bjVnN2pmY0x0Wmdy?= =?utf-8?B?aW1iYWZUUW5hSkEwNlYyTXdSQUFFNmZIaGo4aFJZR3lJcmNiQ2lJQVczOTZQ?= =?utf-8?B?ZzVPMUJRdkVRUEsraUxsNGFpdCtCWEMyVWlEVldxOHBpalFEYkM5SnA3Z2pJ?= =?utf-8?B?NGgvaDZ1SzZjMjZBZXpzWUo5bnNUaVh2bDVuQkdqTFdIdG43OEhHVE5RVnpF?= =?utf-8?B?M0JBSHZyMXBScHdKS3d5WU5NTXJPaWxsckpxM1puOTdVUlpyMUMvVC9RdUg4?= =?utf-8?B?UjRmNXNzc0lNQkJRZDdQc1dlaURscEpwTzJ5bFIvQStUZWV6bVlMalZwaEZh?= =?utf-8?B?M3JYcGthZC82V0NsN2VjTHVhWTR1U3RuZERleDl0Sy9NTUJEMmV0a2Naa0Jr?= =?utf-8?B?SXBuWVlYRGFHNWt0NzJVWTJQVjd5emVvSUlRQnNCZnJxMC9uWE4wR1EzdEZK?= =?utf-8?B?cUhEbVhEOURDNUp2cU9qQjBIVzF6Q2RHRWg5V2szSTJPTnFqbWtUcSs1a0Jl?= =?utf-8?B?bWJjTjgzVHhDQXpWUERoc2pTY1RoNTllR0Y0M3RGcU5YZDQ4QUh2Q1JDa2xH?= =?utf-8?B?OWVkNVAzN0R1L2ZSRGhLOURraVRHR2JSNEs4OXl0bEpKRlk1dStoakU4OFB4?= =?utf-8?B?c0MwMERDVHBoRmxuWUNoZVV5VjA4NE0xMXFBZVJoOGxBZGxNN1FzSG5FeXRJ?= =?utf-8?B?dVl6UlNXZncwOG04cXlMNHd3MXFBdXFDTE1UK2dtZmFzYy9ONktOcjZDQ1Y1?= =?utf-8?B?SHNSaVRXdVJxTTdhZjV6R1IvNWJUTy8wYnRMSFg5dFFHZlRIWHFyTlg5eHo0?= =?utf-8?B?bTdPSmRJRWI5R2FDYnZuVkUyRkFnZEFrUjVTbS9lUC9qWmEzdlVCZWZ0NTlS?= =?utf-8?B?M25aeG1sbDRwb2pJWGVVNWh6ZzVQbXhFUktrTWY1TWI2TG01a2xOR1N6OThP?= =?utf-8?B?UTdTbnNUTG5TVU9GQnBPbzRlY1U1czBhamFaU1FaUm5oeExlR21hYzA4dGU3?= =?utf-8?B?bFJvaGRFQ0hyQUgxNFU2YmJFZTg4OGNOUFBhSThZRmJYSXMycTd3RVRNTmlQ?= =?utf-8?B?Z3ZKMDBNT3Jtb0Q5UXB0U2x6dGhmNitLMit5MWhwMzlQK1o0eVFVNXk5ZTF5?= =?utf-8?B?MFhGV1VNbi9OVy8ycFRYWWd5aDNkTWNSKzJ5VFNvQW1OSzBLQ0xrd2pGMmxo?= =?utf-8?B?bk1rYm5oRUt0UmpWNjMyN1hISzRRWTZKM0xaQkw1bHc1THJOTE1KNUpzYU5K?= =?utf-8?B?enlmaDc1NndUZEFzQW9pU1NxNEg4cW1BczVwZGdMTHBXL2Fta29NZlhrVk1F?= =?utf-8?B?UmRhNUZLZjVsUDZtdm5BREFsVHIvKzZsOFBwK0JZdVpINUp3UVR0VEt1Mmp6?= =?utf-8?B?NHowUE00Yi82RXNqTC80R01IakNqUlprNDJyVnQ5aW1xWDM4d0U3bU1nZFdx?= =?utf-8?B?ZW9WVWNaeVFwZDZuajMycWVzSTFhSHRNdWF0VGxKRVZqYWJMeHorV1lldXhs?= =?utf-8?B?bmZTRkZmQTJRbzZIVitTQWN6V25mTjRQeGRsUUpUbG5rbi9lOUgzTXZPODg3?= =?utf-8?B?bGJ6RTJtRXJuQ2xidFZ0TndxUkwwQ1ZhNnU4UlZqT3FISFl5dHdyUWR4UTR6?= =?utf-8?B?SWluNUlKNzFXcituS1BxMFM2dzZLVHZlU0JWNDVYK04vYmU2UEFYM08vV0xU?= =?utf-8?B?Rk9mK2JtT1ZUOHJkRkliY0tURUNVTUM2djYvT2V6WEZwRnVCYUdoMFNub055?= =?utf-8?B?cUJOTjkyZUNEK1J4SXNid1ljN0lMbERpdW9IUTRIZDFLZTI5WlFhd20vcjBi?= =?utf-8?Q?NIdEwKLK3wuFVtKVgfWx/9I=3D?= X-OriginatorOrg: suse.com X-MS-Exchange-CrossTenant-Network-Message-Id: 47676349-3ed7-4dd0-2313-08d9ab4691a8 X-MS-Exchange-CrossTenant-AuthSource: VI1PR04MB5600.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 19 Nov 2021 10:23:03.0361 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: f7a17af6-1c5c-4a36-aa8b-f5be247aa4ba X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: SX5mjQOE9yUhkhMSa8ZLXZFWIQXMTi3AFJis4LzGHZIxqG1YKxCE5dzGUD/Kotr9ZWnur2XEmWuaWoLTrDdlYg== X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR04MB5168 X-ZohoMail-DKIM: pass (identity @suse.com) X-ZM-MESSAGEID: 1637317401486100001 Content-Type: text/plain; charset="utf-8" From: Lasse Collin It's a more logical place even if the resetting needs to be done only once per LZMA2 stream (if lzma_reset() called in the middle of an LZMA2 stream, .len will already be 0). Link: https://lore.kernel.org/r/20211010213145.17462-4-xiang@kernel.org Signed-off-by: Lasse Collin [Linux commit: a98a25408b0e9b0264abcc3dabfafd9ff2ea1046] Signed-off-by: Jan Beulich Reviewed-by: Luca Fancellu --- a/xen/common/xz/dec_lzma2.c +++ b/xen/common/xz/dec_lzma2.c @@ -791,6 +791,7 @@ static void __init lzma_reset(struct xz_ s->lzma.rep1 =3D 0; s->lzma.rep2 =3D 0; s->lzma.rep3 =3D 0; + s->lzma.len =3D 0; =20 /* * All probabilities are initialized to the same value. This hack @@ -1174,8 +1175,6 @@ XZ_EXTERN enum xz_ret __init xz_dec_lzma } } =20 - s->lzma.len =3D 0; - s->lzma2.sequence =3D SEQ_CONTROL; s->lzma2.need_dict_reset =3D true; =20 From nobody Fri Mar 29 08:33:21 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) client-ip=192.237.175.120; envelope-from=xen-devel-bounces@lists.xenproject.org; helo=lists.xenproject.org; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) smtp.mailfrom=xen-devel-bounces@lists.xenproject.org; arc=pass (i=1 dmarc=pass fromdomain=suse.com); dmarc=pass(p=quarantine dis=none) header.from=suse.com ARC-Seal: i=2; a=rsa-sha256; t=1637317633; cv=pass; d=zohomail.com; s=zohoarc; b=gscxj0eKOkzs143mDn6z9fy1hyrIG2MWRNU4+ji/g1Eg/jnsECeqrkjFDU61lE4yaiH0kcoCWAsz71fdwQItM9q85Bn1rTa1ZOYOO9VJwuiwCAOAtKnvDz8LXCUMrILIyTp7cJz4+JCcKg4QJQqw94nsjYG+/QOhXX7fgsXA8DE= ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1637317633; h=Content-Type:Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:To; bh=s9lhd4h+5NCuGWKWiKjPu07VXVlsfujEz5D+3IHDPrs=; b=FhKVIQ4pyGmpPt+sl7z0OiHilYhqnhdS7D1HBKT7kVO+wRKOMS4n6CNVlyJMbQBQrOdVHBlBtPZWj7TPzx1Nbfgs4i284/OwFbiUeNiPptn3nOYbOG5evLeO/z4Q0+WnLUmhueS54JeHO8+TLuvnivGIYRRtG0j7R429yIRanoo= ARC-Authentication-Results: i=2; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) smtp.mailfrom=xen-devel-bounces@lists.xenproject.org; arc=pass (i=1 dmarc=pass fromdomain=suse.com); dmarc=pass header.from= (p=quarantine dis=none) Return-Path: Received: from lists.xenproject.org (lists.xenproject.org [192.237.175.120]) by mx.zohomail.com with SMTPS id 1637317633665320.5820279475208; Fri, 19 Nov 2021 02:27:13 -0800 (PST) Received: from list by lists.xenproject.org with outflank-mailman.227745.394025 (Exim 4.92) (envelope-from ) id 1mo16c-0005gb-1V; Fri, 19 Nov 2021 10:26:54 +0000 Received: by outflank-mailman (output) from mailman id 227745.394025; Fri, 19 Nov 2021 10:26:54 +0000 Received: from localhost ([127.0.0.1] helo=lists.xenproject.org) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1mo16b-0005gU-UW; Fri, 19 Nov 2021 10:26:53 +0000 Received: by outflank-mailman (input) for mailman id 227745; Fri, 19 Nov 2021 10:26:53 +0000 Received: from se1-gles-flk1-in.inumbo.com ([94.247.172.50] helo=se1-gles-flk1.inumbo.com) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1mo13P-0003FS-QB for xen-devel@lists.xenproject.org; Fri, 19 Nov 2021 10:23:35 +0000 Received: from de-smtp-delivery-102.mimecast.com (de-smtp-delivery-102.mimecast.com [194.104.109.102]) by se1-gles-flk1.inumbo.com (Halon) with ESMTPS id c0845628-4922-11ec-a9d2-d9f7a1cc8784; Fri, 19 Nov 2021 11:23:35 +0100 (CET) Received: from EUR05-AM6-obe.outbound.protection.outlook.com (mail-am6eur05lp2111.outbound.protection.outlook.com [104.47.18.111]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id de-mta-29-V064zvpUOR6KQK9OlhyBEw-1; Fri, 19 Nov 2021 11:23:33 +0100 Received: from VI1PR04MB5600.eurprd04.prod.outlook.com (2603:10a6:803:e7::16) by VI1PR04MB5168.eurprd04.prod.outlook.com (2603:10a6:803:56::33) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4713.22; Fri, 19 Nov 2021 10:23:33 +0000 Received: from VI1PR04MB5600.eurprd04.prod.outlook.com ([fe80::8062:d7cb:ca45:1898]) by VI1PR04MB5600.eurprd04.prod.outlook.com ([fe80::8062:d7cb:ca45:1898%3]) with mapi id 15.20.4713.022; Fri, 19 Nov 2021 10:23:32 +0000 Received: from [10.156.60.236] (37.24.206.209) by AS8P189CA0016.EURP189.PROD.OUTLOOK.COM (2603:10a6:20b:31f::7) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4713.21 via Frontend Transport; Fri, 19 Nov 2021 10:23:31 +0000 X-Outflank-Mailman: Message body and most headers restored to incoming version X-BeenThere: xen-devel@lists.xenproject.org List-Id: Xen developer discussion List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Errors-To: xen-devel-bounces@lists.xenproject.org Precedence: list Sender: "Xen-devel" X-Inumbo-ID: c0845628-4922-11ec-a9d2-d9f7a1cc8784 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.com; s=mimecast20200619; t=1637317414; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=s9lhd4h+5NCuGWKWiKjPu07VXVlsfujEz5D+3IHDPrs=; b=EhAPmzz8mYpSvpOxKtS1UdWNJ/Rx27Mtlv5fl3BcbV4xLOT4n82ZUn2Cd63NHXv9eCMPSV kKCmxqIaIlzgm/23SJ9JS3pw/lzEuPcgPK782+MP7fYFeqOIP0PZf+viEvdtF3tnK6CHR3 cjiRrl+V6f9koA+039U4Q37yg5jNLPM= X-MC-Unique: V064zvpUOR6KQK9OlhyBEw-1 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=gGFx1LVdpmNl+Jq2QE1XbnjByd6FLV+2KG8l93sLFRUiRSuY27yAzBNBziu4xj6chy3HcFtSSMljcR39DgpljkPsRLGnbcNvUZIezT6ebuj/2EsNF6gDsd22ROrLUrpHLHUtuTPXuM2A5f+7CfJjrDvbkQ9lWR9pfa67GsVvb6AFqEgJXuNFROSe0sGyb9DDwwdu5HVRPGZtNLihcMAJLKx++L1E946DCCeYn/STJp/QCu1mNrEGiStkti0iRWqPEpozjkTLzFwiuTA6kzuRFik7+ZyNbMYgccywCRwP1cHXv7SiwFVRlGLIbQc02HHZSwAfQAWigo5B5EcsXVbbgw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=s9lhd4h+5NCuGWKWiKjPu07VXVlsfujEz5D+3IHDPrs=; b=S/b4mYP2+fMjHu7DSlaNcX7nxPIL+UNBg+xkH1hnrNFHGXclmy1/kaqcpM0Wr+Jml2i+sGOsXLl0vgXeaTNMK7ehi5dvfOPGex4KTONl5hXaOUibaXs+G839ZEzUoOkI/Z7AoGT7Yh6Q72I4+myfNow0hmQl3PrH/20Y1xSAGY3ru3aG8HzsIdlw/KtnMLY8kdfXd9Pe/PNYRvLpPhDZG6nYKX9C7Of2erEYfUOFhCGVmHSavRl6B364q8ZtCpgTxA4PEKo0ADd6LmyoeuBj9AmpVqFizrATEgSO7VsmYPdk5Pdf7f1DesBJ/xAZ+UllQDaAAjdQ89iINln/GoEmpg== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=suse.com; dmarc=pass action=none header.from=suse.com; dkim=pass header.d=suse.com; arc=none Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=suse.com; Message-ID: <02555939-5720-af2d-6a0f-550709aab884@suse.com> Date: Fri, 19 Nov 2021 11:23:31 +0100 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:91.0) Gecko/20100101 Thunderbird/91.3.0 Subject: [PATCH 7/7] xz: validate the value before assigning it to an enum variable Content-Language: en-US From: Jan Beulich To: "xen-devel@lists.xenproject.org" Cc: Andrew Cooper , George Dunlap , Ian Jackson , Julien Grall , Stefano Stabellini , Wei Liu References: In-Reply-To: Content-Transfer-Encoding: quoted-printable X-ClientProxiedBy: AS8P189CA0016.EURP189.PROD.OUTLOOK.COM (2603:10a6:20b:31f::7) To VI1PR04MB5600.eurprd04.prod.outlook.com (2603:10a6:803:e7::16) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: ace4981e-2069-4511-42f4-08d9ab46a36d X-MS-TrafficTypeDiagnostic: VI1PR04MB5168: X-Microsoft-Antispam-PRVS: X-MS-Oob-TLC-OOBClassifiers: OLM:4502; X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: 1NHDCqwSE2/ncGNNwjfICdVnUdUbqxGEtoE3ncgftQJmvyWlmKrGv3AHE4Zjrkg0IipN+JOydFOd+E61oRO5ouIW4UfVY78Uk1hTyXokABydLACpUv2Qn7ELkvD5fj5UW8HNxkE4n5dVPaJm2DIXX6yS+IAHqAiClgx6DeCAMtYWg+++xAmtM8/iHtC8u8sLaRKOaQnQT5apSWuUbv5lnPFwGGJZHaKjYHAEGjAay8DMA1XQqLUaca2G0g6ZdxgDhOFwW3xKQZvEoY+6dcc9Qw/+NVgapjj+5BmB/oRe0J3K5r5Iio+0B53IqraeYlzOLxl5hFzDUehuAnQYjpmecqhZcNJThIXAQqq4I+uzWm0xTom78LPKuXowhVAslD/rljWpFIyeMuygt+NcS4DfPSZkiPjmEmxvCBIEUsS2hqZ0g8itH34nE1oUFtsWSt7tcZEsTv5ISflrKPKDJ2v8P5CWYQrCNWM0skTkcQWDJ1gZzKIc9IFFqVka6npJNWKvUP0IN8BIfkdQQcJ2Qi0Oca5TU49hknF9XXUB22jpE7MpXbbF+kjsxFrHzLEEGTx7cg5UC+ZbjU2KoN8G1XBpUnPwxLglXooAUhi3020tZ3dZmwC48nGEhDsTWxxwDTtyTeQtEbz/ROROnr0zAmvEeEyWaa16EEcjMqhVUjpq7fhYV6UPzyGoXWpeUzUdF3Vl6fjFvfamkg04m7AgbqnkQnK4TqfEcRDb20Ya9oYI8ZnaG95hq0PbOpF5TQeDNAK5TZcC3GwQEPl9kqrPNabSxeaMbn8IGM5LJrTOAuxigYceEOzDzVGrMNRSwXgTh3PQ X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:VI1PR04MB5600.eurprd04.prod.outlook.com;PTR:;CAT:NONE;SFS:(366004)(83380400001)(6916009)(38100700002)(66946007)(316002)(6486002)(16576012)(8676002)(966005)(86362001)(31696002)(2906002)(186003)(956004)(5660300002)(2616005)(15650500001)(4326008)(66556008)(54906003)(8936002)(26005)(66476007)(508600001)(31686004)(36756003)(43740500002)(45980500001);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?aTg4SVphREVmeUN6dXROaGRMMUU0Y2dGL2VNMVplL0JhZ2s1dnIwVW1mSjhW?= =?utf-8?B?V3h1RW16NzVGSWFKRDNzSGtFUWVHaGpLUndtNVBJZmVwdkRqUHhsQnBLaHh4?= =?utf-8?B?T1NRQTFOSjNoSVVLREs3SkwrS2ltdzJHYWJHTXFQbjRwZ2prZVVTS1RJdG1X?= =?utf-8?B?SjUyWEZrNTNFNi9HYWhzMVMxNFFMaDdKcFZWUUxqOVVaMEV4T3JaTkFNdDNa?= =?utf-8?B?L0ZXeGxMd3NuaXFxRDVCOXZiSVMwV2xQR0J6V1lDZDRZVTk0NS9MamgxMWli?= =?utf-8?B?ZlZWMTVLOWpiS3lYN2JINjIvWnJHdkNQdEZlNEVEYU5FMlJQS1ZZS1VMaEZC?= =?utf-8?B?U0pCSzV6cldGMTNRZjVkSWhMM2xUcGUrVEJtWGpya25pSFU0U1NRQ1U2S0pE?= =?utf-8?B?KzUyMEtQQnVmY05qVFBZazlUQ1Q1VU1KSHk5T2V2UTZOZno3MlNzWlk4OTNv?= =?utf-8?B?dDFoNkJLcjMzMFpTQk9jTTZVOTlIN2dCRUUrMkg3VDJoZkpmZzM1djNESElZ?= =?utf-8?B?TUk2SEkvdVNQa0ozYVQ3VXFET0JiWHJtQTdzYU5oSUkxZzRSeHpCQVVDSzBQ?= =?utf-8?B?TGtINmNKOXdrdWNHaVc4VE9NL1Z5TEEzVlAwM3ZrRk93UGxBV2I3ZmNBOGg1?= =?utf-8?B?TktoYzVyQ2thdklKaGpmUzN0V1JZRUFtc3Y1OFJlTGMwZnE4RlJORmZaT2p5?= =?utf-8?B?QjVBSmE3THhzaCtXMnFwU2R3Rkd1WTRZRDJ1RkxxdXBwcldtdlpSYWcrYUdi?= =?utf-8?B?MGI4bzhVRGt2Rmt0Z01BVnZBTjU3eTBqTEhxRFhxTHZnaHVRdTU0c1ZQbk5q?= =?utf-8?B?MURTd1FlZ3JYa1Z3RmhvSFlZMGRGS3VWUWpCMVRqeEhSQUNSNXNjSUZBMnNr?= =?utf-8?B?UjBFMS85UTV2UENuWjVad2N4SDljZXIrRXQvajF4YWhOcnAwVzBFK25wVEhB?= =?utf-8?B?MkxJWW9rSk9tWEJiUTRJUWYzWlF2K3llY09MYVM0N2RXY2xNRFZrRlNKTHNY?= =?utf-8?B?dzVLdjY5MEVUbmJZMlptTkJ6T3FaaExmNGZCeVdlQWFlRXFlQy9yVzlWZytE?= =?utf-8?B?WTVMZnZ5YW4wR3ZmNFhqVWpjY2tSNDNxa2c0Mm16UXZGVzRpRThrWVRQN1JH?= =?utf-8?B?RkhEUmRTZ3prSmtKTzhkNHlmWmduMXJoeDNJS05BTWREK282cWZPQllQWVRT?= =?utf-8?B?UmpLb3hGdGl5V3BYT0Eyb2VqVkxhVDBBRTJQL1lIVmlKNjQyVk9KYTFRdWVv?= =?utf-8?B?YTE2Vzkrc0c5YmxhaXQvYkJuY0xhWEszZzVJSkZIN1ljSjlsV2JzQk9QNzc0?= =?utf-8?B?MDdYU2trNGQ5WllHZkNMM3E4a0tvVkl3L1ZIR1hOVEtvbGdCeWxKSk5yeE9T?= =?utf-8?B?L3JhOUtRVnBoSWJLYW94a0VSR1kxRENIRzZ0cTJiNDFiM2FjMXdGWTNNczBB?= =?utf-8?B?MFZpVVJuZmhEcm5lSm5BcFVsV1NEd282QWNlR1Z4VmVqMjIrSmxaN3BzaER2?= =?utf-8?B?WFR4L2JzSE51WmJMNFp6M21iNWQwbzRsM00rbGMvOTVma2V3MEwxNWZvUGUw?= =?utf-8?B?L2NCeGw5TmNJbEwvQmFyUFYrR1h6SmlLMy9zUk1LOVErQmNSNTZvd0g5RkRz?= =?utf-8?B?UXFRVG4vTCtLN1Q4QWZLTldYQW5QNkdjS3BTWWlFZWNsYzhaTC9FWDkrVkE5?= =?utf-8?B?Uy9MVU96djlrRDJoWGY2UGRzQzJYTHI5TlQvVUFkREhscmlIanU5LzZMQ0wz?= =?utf-8?B?MldXMTVrZDdJRis2b3hLS0JyRG9GdkdRdzdrQmtrYUJEdCtsQWp3WjRtZkJm?= =?utf-8?B?MlkwYnhTbXhUUnB1Z3Y4WnhaVER0bjBNbk8xdFgyK2FmNlZLZTJkbnJYZ1Mz?= =?utf-8?B?N2ZlNlJHbTBWbFBRenpVdUhWN3g3WmFRVzV3VmxTM05XRkRlZ0xTS1FrNnZX?= =?utf-8?B?UUZRdFhnSEUzUk9hVU1vRFcwcXYwbEU3WnpLWlp1L2s0dE4wRzBPVjZuVkhz?= =?utf-8?B?MHdTQ1g2NzF6Nks3b2FFdllvKzVLaW41WWhOZ0xtVndrTjdHclJuTEgvdXRS?= =?utf-8?B?UXd2ckpMTzhGVndVNlNpUFdFWG1DV2pjY0R4bERFZzZ6dXF0RFd3ZDA3YU5M?= =?utf-8?B?aEJYTXdyRDhFWlNZQ05wSllFaWZ6dms3NzBXdGk2ZENCMVp0d0ZBeDdqU0hk?= =?utf-8?Q?+el0F70fc5HcwPsHN3j2ru8=3D?= X-OriginatorOrg: suse.com X-MS-Exchange-CrossTenant-Network-Message-Id: ace4981e-2069-4511-42f4-08d9ab46a36d X-MS-Exchange-CrossTenant-AuthSource: VI1PR04MB5600.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 19 Nov 2021 10:23:32.9372 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: f7a17af6-1c5c-4a36-aa8b-f5be247aa4ba X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: iEIpYqLH4tuD6vvJLV8yRUCkF4QIH6Dl2pBuqxh8Imw8u3lnvhlQcrt11UaHCRNYM2tQvZTp6D1DFmrr82O8Rw== X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR04MB5168 X-ZohoMail-DKIM: pass (identity @suse.com) X-ZM-MESSAGEID: 1637317635277100001 Content-Type: text/plain; charset="utf-8" From: Lasse Collin This might matter, for example, if the underlying type of enum xz_check was a signed char. In such a case the validation wouldn't have caught an unsupported header. I don't know if this problem can occur in the kernel on any arch but it's still good to fix it because some people might copy the XZ code to their own projects from Linux instead of the upstream XZ Embedded repository. This change may increase the code size by a few bytes. An alternative would have been to use an unsigned int instead of enum xz_check but using an enumeration looks cleaner. Link: https://lore.kernel.org/r/20211010213145.17462-3-xiang@kernel.org Signed-off-by: Lasse Collin [Linux commit: 4f8d7abaa413c34da9d751289849dbfb7c977d05] Signed-off-by: Jan Beulich Reviewed-by: Luca Fancellu --- a/xen/common/xz/dec_stream.c +++ b/xen/common/xz/dec_stream.c @@ -402,12 +402,12 @@ static enum xz_ret __init dec_stream_hea * we will accept other check types too, but then the check won't * be verified and a warning (XZ_UNSUPPORTED_CHECK) will be given. */ + if (s->temp.buf[HEADER_MAGIC_SIZE + 1] > XZ_CHECK_MAX) + return XZ_OPTIONS_ERROR; + s->check_type =3D s->temp.buf[HEADER_MAGIC_SIZE + 1]; =20 #ifdef XZ_DEC_ANY_CHECK - if (s->check_type > XZ_CHECK_MAX) - return XZ_OPTIONS_ERROR; - if (s->check_type > XZ_CHECK_CRC32) return XZ_UNSUPPORTED_CHECK; #else