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

fix password issue #4

Closed
wants to merge 1 commit into from
Closed

fix password issue #4

wants to merge 1 commit into from

Conversation

clowwindy
Copy link
Contributor

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
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.

3 participants