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

Re: [PATCH] git-cvsserver: handle change type T

From
Martin Langhoff <martin.langhoff@gmail.com>
Date
Mar 17, 2008, 19:34 UTC
Message-ID
<46a038f90803171234m7e3f1cbfgb65311bfb64e6e0d@mail.gmail.com>
In-Reply-To
<7vve3lbwz9.fsf@gitster.siamese.dyndns.org>
On Mon, Mar 17, 2008 at 9:01 PM, Junio C Hamano <gitster@pobox.com> wrote:
>  Martin, likes, dislikes?

Hmmm. Trying to make up my mind on this. It looks like symlink support is broken in cvsserver, it is something we did not even consider, so I am trying to visualise _how_ broken it is, and what it would take to

 - refuse to handle symlinks (temporarily, until we can guarantee our
handling is sane)
 - translate symlinks that are internal - hard, and probably not even
a good idea
 - handle symlinks directly - is the client safe from malicious symlinks?

I have never used symlinks with either git or cvs, so I am unsure as to whether their handling is compatible. Do either of them perform any sanity checks on the targetof the symlink?

cheers,
m
Previous: Junio C HamanoNext: Paolo Bonzini
Message 7 of 8 in “git-cvsserver: handle change type T”
  1. git-cvsserver: handle change type TPaolo Bonzini, Mar 16, 2008
  2. Junio C HamanoMar 16, 2008
  3. Paolo BonziniMar 16, 2008
  4. Junio C HamanoMar 16, 2008
  5. Paolo BonziniMar 17, 2008
  6. Junio C HamanoMar 17, 2008
  7. Martin LanghoffMar 17, 2008
  8. Paolo BonziniMar 17, 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.