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

Fix binja backend stack string detection. Re-enable binja stack strin… #1669

Merged
merged 1 commit into from
Jul 26, 2023
Merged

Conversation

xusheng6
Copy link
Contributor

…g unit test

Checklist

  • No CHANGELOG update needed
  • No new tests needed
  • No documentation update needed

@google-cla
Copy link

google-cla bot commented Jul 25, 2023

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

Copy link
Collaborator

@williballenthin williballenthin left a comment

Choose a reason for hiding this comment

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

lgtm, thanks!

are you able to sign the CLA? based on my understanding, there are no surprises in it, just clarified that contributions can be used in the same spirit as the existing license.

@williballenthin williballenthin added bug Something isn't working binary-ninja labels Jul 25, 2023
@xusheng6
Copy link
Contributor Author

lgtm, thanks!

are you able to sign the CLA? based on my understanding, there are no surprises in it, just clarified that contributions can be used in the same spirit as the existing license.

Jordan tried to sign on behalf of the entire group ([email protected]), but it does not seem to work. I triggered a rescan, but it still list me as not-signed.

@xusheng6
Copy link
Contributor Author

@williballenthin is it possible for you to see what we are missing? We believe we have done everything correct and it is pretty much a dead end for us

@williballenthin
Copy link
Collaborator

ok, thanks for the background.

the CLA process is a new thing that Google requires and i haven't had to debug it yet. i'd prefer not to bother you with it too much, because i recognize that it probably feels like busywork (at least, that's usually how i feel).

let me look around on our side and see if we can figure out what's going on. i'll reach out with any clarifying questions.

note that, unfortunately, i'm afk for this week so i might be a bit delaying in finishing this. github mobile from the hotel room is a marvel but not a replacement for a real dev environment!

@xusheng6
Copy link
Contributor Author

ok, thanks for the background.

the CLA process is a new thing that Google requires and i haven't had to debug it yet. i'd prefer not to bother you with it too much, because i recognize that it probably feels like busywork (at least, that's usually how i feel).

let me look around on our side and see if we can figure out what's going on. i'll reach out with any clarifying questions.

note that, unfortunately, i'm afk for this week so i might be a bit delaying in finishing this. github mobile from the hotel room is a marvel but not a replacement for a real dev environment!

No hurry, please take your time!

@xusheng6
Copy link
Contributor Author

@williballenthin The CLA thing now works. In fact, after we sign the CLA, it needs to be approved by Google. And Jordan got an approval email this morning. After that, I rescan the PR and I am listed as already signed.

@williballenthin
Copy link
Collaborator

great to hear!

based on what i read, Google ensures that an appropriate representative signs the CLA for the company. ie, a lawyer or executive and not an intern. so, maybe that review of titles took the little extra time.

thanks for your patience working through the process, and for the PR!

@williballenthin williballenthin merged commit dd53349 into mandiant:master Jul 26, 2023
23 of 25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
binary-ninja bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants