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

Re: [PATCH 3/5] git-cvsserver: take care of empty passwords

From
ÁLÁshin László <ashinlaszlo@gmail.com>
Date
Jul 6, 2010, 18:21 UTC
Message-ID
<AANLkTinkg-i3KczWliWymP1FWYx0_YeoLGLHq2h2jLa-@mail.gmail.com>
In-Reply-To
<AANLkTilSvEf6DIM_JZql8pTnSCrnFQ2tAOGuqownVl4e@mail.gmail.com>

descramble() dies if it is called with empty string as an argument. This patch only fixes this.

The test case written to check empty passwords only checks the case when the user is not in the authdb, so the old code doesn't even call descramble().

On Tue, Jul 6, 2010 at 20:05, Ævar Arnfjörð Bjarmason <avarab@gmail.com> wrote:
Show 9 quoted lines
> On Tue, Jul 6, 2010 at 17:37, Áshin László <ashinlaszlo@gmail.com> wrote:
>> Do not try to descramble them.
>
> This commit message isn't very revealing. What does this do exactly?
> Is the behavior with empty passwords changed now? Does git-cvsserver
> no longer support empty passwords? Is the on-disk format in the authdb
> just different? (I.e. "" instead of "A" or something). Or something
> else?
>
Previous: Ævar Arnfjörð Bjarmason
Message 3 of 3 in “git-cvsserver: take care of empty passwords”
  1. 3/5 git-cvsserver: take care of empty passwordsÁshin László, Jul 6, 2010
  2. Ævar Arnfjörð BjarmasonJul 6, 2010
  3. Áshin LászlóJul 6, 2010

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.