Re: [PATCH v2 01/22] builtin/ls-remote: plug leaking server options

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Subject: Re: [PATCH v2 01/22] builtin/ls-remote: plug leaking server options
From: "Kristoffer Haugsbakk" <kristofferhaugsbakk@xxxxxxxxxxxx>
Date: Mon, 04 Nov 2024 23:18:23 +0100
Cc: git@xxxxxxxxxxxxxxx, "Taylor Blau" <me@xxxxxxxxxxxx>, "Toon Claes" <toon@xxxxxxxxx>
Feedback-id: i8b11424c:Fastmail
In-reply-to: <qxtstkkch5toz6t7jdknwkdzrlkfu3b7gtw3wpg2iuqsmliqlq@w2qz65ujwj4r>
References: <cover.1728624670.git.ps@pks.im> <cover.1729502823.git.ps@pks.im> <89b66411354437a1e3a97751f185889dffb84126.1729502824.git.ps@pks.im> <qxtstkkch5toz6t7jdknwkdzrlkfu3b7gtw3wpg2iuqsmliqlq@w2qz65ujwj4r>

On Mon, Nov 4, 2024, at 23:10, Justin Tobler wrote:
> On 24/10/21 11:28AM, Patrick Steinhardt wrote:
>> The server options populated via `OPT_STRING_LIST()` is never cleared,
>
> s/is/are/

I guess “is” was chosen because “the list is”.

   The [list] populated via `OPT_STRING_LIST()` is never cleared,

-- 
Kristoffer Haugsbakk

References:
- [PATCH 00/21] Memory leak fixes (pt.9)
  - From: Patrick Steinhardt
- [PATCH v2 00/22] Memory leak fixes (pt.9)
  - From: Patrick Steinhardt
- [PATCH v2 01/22] builtin/ls-remote: plug leaking server options
  - From: Patrick Steinhardt
- Re: [PATCH v2 01/22] builtin/ls-remote: plug leaking server options
  - From: Justin Tobler

Prev by Date: Re: [PATCH v2 07/22] pretty: clear signature check
Next by Date: Re: [PATCH v2 09/22] builtin/commit: fix leaking change data contents
Previous by thread: Re: [PATCH v2 01/22] builtin/ls-remote: plug leaking server options
Next by thread: [PATCH v2 02/22] t/helper: fix leaks in "reach" test tool
Index(es):
- Date
- Thread

[Index of Archives] [Linux Kernel Development] [Gcc Help] [IETF Annouce] [DCCP] [Netdev] [Networking] [Security] [V4L] [Bugtraq] [Yosemite] [MIPS Linux] [ARM Linux] [Linux Security] [Linux RAID] [Linux SCSI] [Fedora Users]