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

[Postgresql] Initial integration, new fuzzer added #4058

Merged
merged 15 commits into from
Jul 17, 2020

Conversation

yooyoo9
Copy link
Contributor

@yooyoo9 yooyoo9 commented Jul 1, 2020

  • Added Postgresql integration files
  • Added new fuzzer for the parser

cc: @sfrost

@yooyoo9
Copy link
Contributor Author

yooyoo9 commented Jul 1, 2020

@sfrost Would it be possible to put the fuzzers in the upstream repo? (Apply patch add_fuzzers.diff)

@inferno-chromium
Copy link
Collaborator

Please ping us for review once developer's comments are all addressed.

Copy link
Collaborator

@inferno-chromium inferno-chromium left a comment

Choose a reason for hiding this comment

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

Requesting changes as per developer's review.

@yooyoo9
Copy link
Contributor Author

yooyoo9 commented Jul 17, 2020

@inferno-chromium developer's comments all addressed.

@yooyoo9
Copy link
Contributor Author

yooyoo9 commented Jul 17, 2020

@inferno-chromium changes made, currently waiting to see if changes in add_fuzzers.diff are allowed to go upstrream

@inferno-chromium
Copy link
Collaborator

@inferno-chromium changes made, currently waiting to see if changes in add_fuzzers.diff are allowed to go upstrream

You can remove that diff file in another cl. atleast project will start getting fuzzed.

@inferno-chromium inferno-chromium merged commit b189f86 into google:master Jul 17, 2020
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 this pull request may close these issues.

3 participants