Skip to content
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

Segmentation fault when using incorrect CLI arguments #16

Open
aminya opened this issue Apr 22, 2024 · 0 comments · May be fixed by #17
Open

Segmentation fault when using incorrect CLI arguments #16

aminya opened this issue Apr 22, 2024 · 0 comments · May be fixed by #17

Comments

@aminya
Copy link

aminya commented Apr 22, 2024

I tried to use the CLI via these arguments as I didn't know what the expected inputs because of lack of an example or docs are.

./json2cpp test ./test.json ./test.cpp

It seems the third argument is incorrect as it should be a name. The arguments need to be validated to prevent segmentation faults. Also, it would be useful to have more documentation.

The correct way to use the CLI is

./json2cpp test ./test.json test
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant