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

hmac.cpp:68:11: error: aggregate ‘HMAC_CTX hmac’ has incomplete type and cannot be defined #5

Closed
OlafvdSpek opened this issue Mar 23, 2018 · 2 comments

Comments

@OlafvdSpek
Copy link
Contributor

OlafvdSpek commented Mar 23, 2018

libssl-dev:amd64                       1.1.0f-3+deb9u1

olaf@debian:~/josepp/build$ make
Scanning dependencies of target josepp-static
[  9%] Building CXX object CMakeFiles/josepp-static.dir/src/tools.cpp.o
[ 18%] Building CXX object CMakeFiles/josepp-static.dir/src/claims.cpp.o
[ 27%] Building CXX object CMakeFiles/josepp-static.dir/src/crypto.cpp.o
[ 36%] Building CXX object CMakeFiles/josepp-static.dir/src/hmac.cpp.o
/home/olaf/josepp/src/hmac.cpp: In member function ‘virtual std::__cxx11::string jose::hmac::sign(const string&)’:
/home/olaf/josepp/src/hmac.cpp:68:11: error: aggregate ‘HMAC_CTX hmac’ has incomplete type and cannot be defined
  HMAC_CTX hmac;
           ^~~~
/home/olaf/josepp/src/hmac.cpp:69:21: error: ‘HMAC_CTX_init’ was not declared in this scope
  HMAC_CTX_init(&hmac);
                     ^
/home/olaf/josepp/src/hmac.cpp:77:24: error: ‘HMAC_CTX_cleanup’ was not declared in this scope
  HMAC_CTX_cleanup(&hmac);

https://www.openssl.org/docs/man1.1.0/crypto/HMAC_CTX_free.html

troian added a commit that referenced this issue Mar 23, 2018
Change-Id: Iaccbdab208693aa53e200caab6b4874aff67aa5d
Signed-off-by: Artur Troian <[email protected]>
@troian
Copy link
Owner

troian commented Mar 23, 2018

@OlafvdSpek can you check referenced commit plz. Don't have OpenSSL 1.1.0 on quick look

troian added a commit that referenced this issue Mar 23, 2018
Change-Id: I832770a31f5ba998b3912af2ab712ec9d5bca391
Signed-off-by: Artur Troian <[email protected]>
@troian
Copy link
Owner

troian commented Mar 23, 2018

Tnx for reporting. Fix merged

@troian troian closed this as completed Mar 23, 2018
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

No branches or pull requests

2 participants