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

sha1_update function name conflict #28

Closed
ymantijunk opened this issue Jun 10, 2013 · 6 comments
Closed

sha1_update function name conflict #28

ymantijunk opened this issue Jun 10, 2013 · 6 comments
Assignees

Comments

@ymantijunk
Copy link

This is not an issue, just a suggestion to change the function name "sha1_update()" and "sha1()" in "crypto/hash/sha1.c" as it conflicts with a system library.

Thanks for the wonderful soffware.
YM

@hasufell
Copy link

hasufell commented Oct 9, 2013

linphone upstream seems to have kinda forked libsrtp because of this? That makes it extremely difficult for packagers to support srtp in the linphone provided ortp library. Not really your problem, but if this can fix it...

@cmcqueen
Copy link

cmcqueen commented Jan 5, 2014

What's the status with this? linphone's fork of libsrtp is difficult to manage from a package management point-of-view. That is, linphone is relying on a version of libsrtp that has no assignable version number. Can this issue be fixed?

@nuqleo
Copy link

nuqleo commented Jan 9, 2014

There is yet one renaming on git://git.linphone.org/srtp.git repo aes_encrypt to srtp_aes_encrypt to avoid conflict with libzrtpcpp.

@mgorny
Copy link

mgorny commented Sep 13, 2014

Is sha1_update() even needed as a part of public API? Maybe it would be easier to make it static, or give it hidden visibility.

@jfigus
Copy link
Contributor

jfigus commented Sep 22, 2014

I agree that sha1_update() shouldn't be part of the public API. IMHO, none of the crypto/include header files should be published. Unfortunately since sha1_update() is currently in one of the published header files, removing it creates the risk of breaking downstream applications.

@jfigus jfigus self-assigned this Nov 4, 2014
@jfigus
Copy link
Contributor

jfigus commented Nov 4, 2014

This is resolved by c270245 in the 2_0_0_dev branch.

@jfigus jfigus closed this as completed Nov 4, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants