Skip to content
This repository was archived by the owner on Apr 24, 2022. It is now read-only.

fix password issue#4

Closed
clowwindy wants to merge 1 commit intodndx:masterfrom
clowwindy:master
Closed

fix password issue#4
clowwindy wants to merge 1 commit intodndx:masterfrom
clowwindy:master

Conversation

@clowwindy
Copy link
Copy Markdown

getopt should be used with strcpy, or each optarg will be overriden.

I didn't fix other args, you may check them as well.

@dndx
Copy link
Copy Markdown
Owner

dndx commented Jan 4, 2013

The problem here is not getopt(3), getopt(3) always return pointer to argv so that should not be a problem.

The real problem is that we used uv_setup_args() to override system argv in order to modify ps output. So the original pointer returned by getopt(3) are not pointing to valid values.

This issue should be fixed in commit 037fed9 , according to my test, it's working well now.

@dndx dndx closed this Jan 4, 2013
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.

2 participants