Received: from malur.postgresql.org ([217.196.149.56]) by arkaria.postgresql.org with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.94.2) (envelope-from ) id 1r1Ze8-00C3YW-HL for pgsql-hackers@arkaria.postgresql.org; Fri, 10 Nov 2023 22:06:36 +0000 Received: from localhost ([127.0.0.1] helo=malur.postgresql.org) by malur.postgresql.org with esmtp (Exim 4.94.2) (envelope-from ) id 1r1Ze6-005ZqZ-Re for pgsql-hackers@arkaria.postgresql.org; Fri, 10 Nov 2023 22:06:34 +0000 Received: from makus.postgresql.org ([2001:4800:3e1:1::229]) by malur.postgresql.org with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.94.2) (envelope-from ) id 1r1Ze6-005ZqQ-Fs for pgsql-hackers@lists.postgresql.org; Fri, 10 Nov 2023 22:06:34 +0000 Received: from mail-ed1-x52d.google.com ([2a00:1450:4864:20::52d]) by makus.postgresql.org with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256 (Exim 4.94.2) (envelope-from ) id 1r1Zdz-004y6W-N9 for pgsql-hackers@lists.postgresql.org; Fri, 10 Nov 2023 22:06:33 +0000 Received: by mail-ed1-x52d.google.com with SMTP id 4fb4d7f45d1cf-544455a4b56so4065565a12.1 for ; Fri, 10 Nov 2023 14:06:27 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1699653986; x=1700258786; darn=lists.postgresql.org; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:from:to:cc:subject:date :message-id:reply-to; bh=u0ZVPIngqYXpigclTuxOCziHczLhyeZ1l70bTl7zY58=; b=OzACIiAUGzcwbwCejmnMT8VGNb5xVMFij7oxDh1RCHlbcBfZ+tUoUqtvEs6LZAd8G/ Y1cdYd22JU7gLsLJlZ+KVWqBmCuxf0WPDJ/pes8NKc2nkvJJXcYFUp6kAdy3bib7AMTi X9MtvDQAOWdrzYrK+kzhuL5LXGkX7dUkXg/zmXCKyzLXh7pxxM2un5ERQ03Fle/vug1l lypPB5EDkvwV9ZLGsXI8YIa9Bcf11slLe4VdiWMxVtTBjZSBggdEXfzXn7d4Hjv63Vx6 EbgsTawM3EQfr/fvVuABP72ZcyjzGQI5zooo4ybYIuG/Q6FQ1qN/aKwJ9kfEMdU8WqiT 90TQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1699653986; x=1700258786; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=u0ZVPIngqYXpigclTuxOCziHczLhyeZ1l70bTl7zY58=; b=W9xGeXgP7r2CooIhPA3YhVaU3o/7F3+rsnZg6s2aaUeUsS3iRAjrSsbss+Uw2yaCSm T8ZCOWFm/oy3GkRwqNRw/5JDyu3AxcTjqRqf9lhe+FaQuiohv4S5mYa1Lx/4FyelODx9 7d70gOqUkTuOkmozo9JFT+sq6cjgWhvPBdrEpaKS43kwzSwXB2eqVRPCMh9myC6otYJT QDf/N55b3a1NNhzrj3nvV52mSNbaYEPmKJENlfWM/EbjkPRQwjhGiMHJByzmWF8vocEf vs0onYPUPcgoedMjX5vM5Vlqu/LjoHr7KRueZOmZeA2UlGFHCKzCncQ+TuB/xEz3Ij/d l4dw== X-Gm-Message-State: AOJu0YzNuo5Mdv+aFNVOLuH6Jjo70arbdBS3ln9UV0MdrB9CaPL3r7gS etO8mejMEy2DNGQXy/vC0xfBm3bIkXuvykD1zHk= X-Google-Smtp-Source: AGHT+IH3wK/kueEbaRJd6kHYKe/6/C1e6Sp3sPkHw+HAukgvGkDNFYITPG18JH6Mt1BkamXRZ61xNirkzYQ3d1jRP7E= X-Received: by 2002:a05:6402:35c8:b0:533:c55f:5830 with SMTP id z8-20020a05640235c800b00533c55f5830mr466974edc.28.1699653985487; Fri, 10 Nov 2023 14:06:25 -0800 (PST) MIME-Version: 1.0 References: In-Reply-To: From: Thomas Munro Date: Sat, 11 Nov 2023 11:05:48 +1300 Message-ID: Subject: Re: pgsql: Don't trust unvalidated xl_tot_len. To: Christoph Berg Cc: Thomas Munro , pgsql-hackers@lists.postgresql.org Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable List-Id: List-Help: List-Subscribe: List-Post: List-Owner: List-Archive: Archived-At: Precedence: bulk On Sat, Nov 11, 2023 at 10:42=E2=80=AFAM Christoph Berg w= rote: > > I haven't investigated the details yet, and it's not affecting the > > builds on apt.postgresql.org, but the Debian amd64 and i386 regression > > tests just failed this test on PG13 (11 and 15 are ok): > > 12 and 14 are also failing, now on Debian unstable. (Again, only in > the salsa.debian.org tests, not on apt.postgresql.org's buildds.) > > 12 amd64: https://salsa.debian.org/postgresql/postgresql/-/jobs/4898857 > 12 i386: https://salsa.debian.org/postgresql/postgresql/-/jobs/4898858 > > The tests there are running in Docker containers. Hmm. regress_log_039_end_of_wal says: No such file or directory at /builds/postgresql/postgresql/debian/output/source_dir/build/../src/test/pe= rl/TestLib.pm line 655. In the 13 branch we see that's in the new scan_server_header() subroutine where it tries to open the header, after asking pg_config --includedir-server where it lives. Hmm...