Skip to content
This repository was archived by the owner on Jul 23, 2024. It is now read-only.

HAWQ-1729.fix memory wrong usage#1458

Closed
ghost wants to merge 1 commit intomasterfrom
unknown repository
Closed

HAWQ-1729.fix memory wrong usage#1458
ghost wants to merge 1 commit intomasterfrom
unknown repository

Conversation

@ghost
Copy link
Copy Markdown

@ghost ghost commented Jul 23, 2019

No description provided.

@huor huor requested review from huor and wcl14 July 23, 2019 02:49
@huor huor assigned ghost Jul 23, 2019
* ident_unix(), which causes memory freed referenced by pw
*/

local_name[IDENT_USERNAME_MAX + 1] = 0;
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

local_name[IDENT_USERNAME_MAX] = '\0';

@huor
Copy link
Copy Markdown
Contributor

huor commented Jul 23, 2019

LGTM +1

@huor huor self-requested a review July 23, 2019 03:00
@wcl14
Copy link
Copy Markdown
Contributor

wcl14 commented Jul 23, 2019

LGTM

@ghost ghost closed this Jul 23, 2019
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants