Re: [PATCH 2/4] reftable/merged: fix zero-sized allocation when there are no readers

From: Kristoffer Haugsbakk <kristofferhaugsbakk@fastmail.com>
Date: Dec 21, 2024, 12:40 UTC
Message-ID: <b4ba502a-b689-4736-9712-443a137b5d2c@app.fastmail.com>
In-Reply-To: <20241221-b4-pks-reftable-oom-fix-without-readers-v1-2-12db83a3267c@pks.im>

Hi

On Sat, Dec 21, 2024, at 12:50, Patrick Steinhardt wrote:

> It was reported [1c

A Neo user, I see.

s/c/]/

> [snip]
> the result. Now that we do we notice and thus return an out-of-memory
> error to the caller.

s/Now that we do we/Now we do notice/

Repeat of “we”. And “that” can’t be be used since it has nothing to connect to.

-- 
Kristoffer Haugsbakk

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.