Skip to content

Conversation

@dscho
Copy link
Member

@dscho dscho commented Nov 17, 2025

This patch wants to be upstreamed...

@dscho dscho self-assigned this Nov 17, 2025
@gitgitgadget
Copy link

gitgitgadget bot commented Nov 17, 2025

There are issues in commit 72938e8:
wincred: Avoid memory corruption
Prefixed commit message must be in lower case

`wcsncpy_s()` wants to write the terminating null character so we need
to allocate one more space for it in the target memory block.

This should fix crashes when trying to read passwords.  When this
happened, the password/token wouldn't print out and Git would therefore
ask for a new password every time.

Signed-off-by: David Macek <david.macek.0@gmail.com>
Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
@dscho dscho force-pushed the wincred-Avoid-memory-corruption-5856- branch from 72938e8 to 4fdcccc Compare November 17, 2025 20:31
@dscho
Copy link
Member Author

dscho commented Nov 17, 2025

/submit

@gitgitgadget
Copy link

gitgitgadget bot commented Nov 17, 2025

Submitted as pull.2005.git.1763411984773.gitgitgadget@gmail.com

To fetch this version into FETCH_HEAD:

git fetch https://github.com/gitgitgadget/git/ pr-2005/dscho/wincred-Avoid-memory-corruption-5856--v1

To fetch this version to local tag pr-2005/dscho/wincred-Avoid-memory-corruption-5856--v1:

git fetch --no-tags https://github.com/gitgitgadget/git/ tag pr-2005/dscho/wincred-Avoid-memory-corruption-5856--v1

@gitgitgadget
Copy link

gitgitgadget bot commented Nov 17, 2025

This patch series was integrated into seen via git@2f886f0.

@gitgitgadget gitgitgadget bot added the seen label Nov 17, 2025
@gitgitgadget
Copy link

gitgitgadget bot commented Nov 18, 2025

This patch series was integrated into seen via git@7d67e77.

@gitgitgadget
Copy link

gitgitgadget bot commented Nov 18, 2025

This branch is now known as js/wincred-get-credential-alloc-fix.

@gitgitgadget
Copy link

gitgitgadget bot commented Nov 18, 2025

This patch series was integrated into seen via git@6a1b73d.

@gitgitgadget
Copy link

gitgitgadget bot commented Nov 19, 2025

This patch series was integrated into seen via git@37b073b.

@gitgitgadget
Copy link

gitgitgadget bot commented Nov 19, 2025

This patch series was integrated into next via git@4c43c53.

@gitgitgadget gitgitgadget bot added the next label Nov 19, 2025
@gitgitgadget
Copy link

gitgitgadget bot commented Nov 20, 2025

There was a status update in the "New Topics" section about the branch js/wincred-get-credential-alloc-fix on the Git mailing list:

Under-allocation fix.

Will merge to 'master'.
source: <pull.2005.git.1763411984773.gitgitgadget@gmail.com>

@gitgitgadget
Copy link

gitgitgadget bot commented Nov 20, 2025

This patch series was integrated into seen via git@e6f3233.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants