Received: by 2002:a05:6359:c8b:b0:c7:702f:21d4 with SMTP id go11csp2748170rwb; Sat, 8 Oct 2022 13:35:17 -0700 (PDT) X-Google-Smtp-Source: AMsMyM5nL5QlmYRnF9QH/J9pRyM9DYZ2lTAX361VrWzSA4qPAUEvVnWu+IOaDQTw0dnFfCazn8jK X-Received: by 2002:a17:902:cec9:b0:178:5fa6:f3c3 with SMTP id d9-20020a170902cec900b001785fa6f3c3mr10978377plg.83.1665261317498; Sat, 08 Oct 2022 13:35:17 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1665261317; cv=none; d=google.com; s=arc-20160816; b=0JEZoUiaGq5oDzCk/erkfWKocYCDoTtl2CrcclPXx0qjkUnecoVaynmlVxuorCjXRP AidUm72BgPCYqwQEo1DtHg95DNpc8H5s/qAVUfBnXu6tmV0tcddfOfIgiU1fMdZsF0S+ OSGX+dTQYffRDgsZtNGxQXibcRpLuB4Szec5hNISHkQFX4d1ru9PwlIB0hNoRggrY7Kg /7T/BSaWZbi7ZRFq66piVVSPG/ihGveT3/FnZPFAyB4U1PrFCZcRJi8njrT30qvsjWDB HAf62yA0RsfjsTDBtxqATRadBHZ1yrOY71O7Wi1agtr3vAmsZF149DN1OL22vUOOZQvd omEg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:in-reply-to:content-disposition:mime-version :references:message-id:subject:cc:to:from:date:dkim-signature; bh=mAgGQiYibt6/W5CAo+9L5oS3sdpJlLe5MrCeTey7nag=; b=qNm8xpG8l748yvMgRy3+UjoZ/LuyefHUVuiHz5u3RiwEUacfGpeJUbHAvrEWonj0WP AyYJp/G3rcaWgSIjEZf7UTXXZUN8vlosplCAhK/hfRb1aG/hLfzlPUvR2HJW0K9vSk9Y R6VDg2UUZ+UiSRj1NG1x037oncEoCfLnWXAWb3wkJPJEN1ZLnxSzMKl83fL/DdLVXnio SIqxfHqbPM4IA+tgXJEB+E5YBmojG1zC5eBFqVQ4J/aerq1HVoCYDXxqmlvekOLjB5AF 8dhqxIGrLqQ/j2rEOf177RaL3GkjK0aKNKmWBZj4cimhqbIuBkYDi3ETafrXFWgfIZ+f ODcQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@chromium.org header.s=google header.b=J+8YFK6i; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=chromium.org Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id i16-20020a170902cf1000b00178af7f1832si6744584plg.216.2022.10.08.13.35.05; Sat, 08 Oct 2022 13:35:17 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@chromium.org header.s=google header.b=J+8YFK6i; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=chromium.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229831AbiJHTo2 (ORCPT + 99 others); Sat, 8 Oct 2022 15:44:28 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:49544 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229597AbiJHTo0 (ORCPT ); Sat, 8 Oct 2022 15:44:26 -0400 Received: from mail-pj1-x1031.google.com (mail-pj1-x1031.google.com [IPv6:2607:f8b0:4864:20::1031]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 0B7532EF04 for ; Sat, 8 Oct 2022 12:44:26 -0700 (PDT) Received: by mail-pj1-x1031.google.com with SMTP id v10-20020a17090a634a00b00205e48cf845so9943050pjs.4 for ; Sat, 08 Oct 2022 12:44:26 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=chromium.org; s=google; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc:subject:date:message-id:reply-to; bh=mAgGQiYibt6/W5CAo+9L5oS3sdpJlLe5MrCeTey7nag=; b=J+8YFK6iUM86zYLhmFX3+t7qRzul/K4tXXW0089HqM4aCs4d/CCqPjf7/fv65zexgD XdhVVzx+W1tmmk8HUs48KiXHOjR0EtbRpYzJxa511QAN+oE4KpqYCS7blqqBQAVGwfYJ iQ5bB4mISKlyHvN15qjmcYHOxWgwHQyArJVN8= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=mAgGQiYibt6/W5CAo+9L5oS3sdpJlLe5MrCeTey7nag=; b=2OFwvwbD8vMJumx42ukjitS0cQwH+tD928x91/hizoohyN/5YQBs/Kf5t49HyoTKh+ XQXmezCZ8qz1t5GzPyvAiKr1e7i+mljzEmN8/TGy4zHv7hzwBbLG8+jO9QlTNdJFE4Mi AuzofVIFLFQbO3CyZSd8xcv4wSpR6ztQaEm6TAHT2Pge2sbZT3TYLxI4d8ZkE6cCMLFk VuFaFoVyw19z4FEGqhQMgb+vzdQw0GLYTJV1T9yGT4hPGMoFhTuC1s7CkhbzpeC9Yvy+ iqiosIB0UnOTfdBdY91qerTf0pRA34ehzHncxUtzS0k8569pzNFgvjyYaSexyJ7DKXOO WMxQ== X-Gm-Message-State: ACrzQf1jhwxYJ3LRomTn8+dRbVpAljbhM2rc03TpyHEzJRgFtRyZIhOI rllgR5Yl9tpGD3hde17Ows5Rlg== X-Received: by 2002:a17:902:e808:b0:17f:92cb:2fdd with SMTP id u8-20020a170902e80800b0017f92cb2fddmr7914995plg.137.1665258265543; Sat, 08 Oct 2022 12:44:25 -0700 (PDT) Received: from www.outflux.net (smtp.outflux.net. [198.145.64.163]) by smtp.gmail.com with ESMTPSA id b1-20020a17090a100100b002005fcd2cb4sm6673353pja.2.2022.10.08.12.44.24 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sat, 08 Oct 2022 12:44:24 -0700 (PDT) Date: Sat, 8 Oct 2022 12:44:23 -0700 From: Kees Cook To: Ard Biesheuvel Cc: "Guilherme G. Piccoli" , linux-kernel@vger.kernel.org, linux-fsdevel@vger.kernel.org, kernel-dev@igalia.com, kernel@gpiccoli.net, anton@enomsg.org, ccross@android.com, tony.luck@intel.com Subject: Re: [PATCH 5/8] pstore: Fix long-term implicit conversions in the compression routines Message-ID: <202210081242.0B0F8B7@keescook> References: <20221006224212.569555-1-gpiccoli@igalia.com> <20221006224212.569555-6-gpiccoli@igalia.com> <202210061634.758D083D5@keescook> <202210071234.D289C8C@keescook> <11e03e8d-7711-330d-e0d4-808ef9acec3a@igalia.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE, SPF_HELO_NONE,SPF_PASS autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Sat, Oct 08, 2022 at 07:52:48PM +0200, Ard Biesheuvel wrote: > Again, please correct me if I am missing something here (Kees?). Are > there cases where we compress data that may be compressed already? Nope, for dmesg, it should all only be text. I'd agree -- the worst case seems a weird thing to need. -- Kees Cook