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

Added AES-128/192/256 with ECB mode to rahash2 #4402

Closed
wants to merge 2 commits into from
Closed

Conversation

l4l
Copy link
Contributor

@l4l l4l commented Mar 23, 2016

#4254 related

There is still some work for IV-family mode AESes like CBC/CFB/OFB

@radare
Copy link
Collaborator

radare commented Mar 23, 2016

@l4l
Copy link
Contributor Author

l4l commented Mar 23, 2016

That's really strange. I can't reproduce it with clang/gcc with travis script and this piece of code link properly.
Any ideas?

@Maijin
Copy link
Contributor

Maijin commented Mar 23, 2016

@radare
Copy link
Collaborator

radare commented Mar 23, 2016

You can reproduce it on mac or mingw or android static builds.

There are scripts in sys /for all this stuff

The thing is that linux linker is quite lazy and solves this dependency at runtime while pther systems require explicit compile time linkage

On 23 Mar 2016, at 14:48, Evgeny [email protected] wrote:

That's really strange. I can't reproduce it with clang/gcc and this piece of code link properly.
Any ideas?


You are receiving this because you commented.
Reply to this email directly or view it on GitHub

@radare
Copy link
Collaborator

radare commented Mar 23, 2016

Use tabs, not spaces

@radare
Copy link
Collaborator

radare commented Mar 23, 2016

Anyway, i did the fixes and squashed your commits.

Thanks! :D

@radare radare closed this Mar 23, 2016
@radare radare added this to the 0.10.2 milestone Mar 23, 2016
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