git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH maint] builtin-merge.c: fix memory under-allocation

From
Miklos Vajna <vmiklos@frugalware.org>
Date
Oct 9, 2008, 00:17 UTC
Message-ID
<20081009001727.GP536@genesis.frugalware.org>
In-Reply-To
<ZuhLzndR5Uvj7-_NyasP-FHF3AqTyLx-0sjNHMd0BC3LXC8LdLqBeQ@cipher.nrlssc.navy.mil>
On Wed, Oct 08, 2008 at 07:07:54PM -0500, Brandon Casey <casey@nrlssc.navy.mil> wrote:
> While we're at it, change the allocation to reference the variable it is
> allocating memory for to try to prevent a similar mistake if the type is
> changed in the future.

If this is really a problem, then I think it would be good to mention this in Documentation/CodingGuidelines.

Previous: Brandon CaseyNext: Brandon Casey
Message 2 of 7 in “builtin-merge.c: fix memory under-allocation”
  1. builtin-merge.c: fix memory under-allocationBrandon Casey, Oct 9, 2008
  2. Miklos VajnaOct 9, 2008
  3. Brandon CaseyOct 9, 2008
  4. Junio C HamanoOct 9, 2008
  5. Brandon CaseyOct 9, 2008
  6. Miklos VajnaOct 9, 2008
  7. Shawn O. PearceOct 9, 2008

Read the whole thread, see it on lore, or plain text.

$ 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.