-
Notifications
You must be signed in to change notification settings - Fork 158
wincred: Avoid memory corruption #2005
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
wincred: Avoid memory corruption #2005
Conversation
|
There are issues in commit 72938e8: |
`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>
72938e8 to
4fdcccc
Compare
|
/submit |
|
Submitted as pull.2005.git.1763411984773.gitgitgadget@gmail.com To fetch this version into To fetch this version to local tag |
|
This patch series was integrated into seen via git@2f886f0. |
|
This patch series was integrated into seen via git@7d67e77. |
|
This branch is now known as |
|
This patch series was integrated into seen via git@6a1b73d. |
|
This patch series was integrated into seen via git@37b073b. |
|
This patch series was integrated into next via git@4c43c53. |
|
There was a status update in the "New Topics" section about the branch Under-allocation fix. Will merge to 'master'. source: <pull.2005.git.1763411984773.gitgitgadget@gmail.com> |
|
This patch series was integrated into seen via git@e6f3233. |
This patch wants to be upstreamed...