tarantool / http

79%

master: 79%

LAST BUILD ON BRANCH fix-ffi-type-declaration

branch: fix-ffi-type-declaration

CHANGE BRANCH

x

Reset

• fix-ffi-type-declaration
• 0x501D/add_tarantool_version
• 0x501D/bump_version
• 0x501D/ci-enable-tests
• 0x501D/ci_enable_tests
• 0x501D/ci_fix_ubuntu
• 0x501D/ci_remove_packpack_crutch
• 0x501D/packaging
• 0x501D/release-1.3.0
• 0x501D/remove_coverage
• DerekBum/TNTP-1058-update-ubuntu-runners
• DerekBum/gh-196-add-http-role
• artembo/packaging
• check_packpack_fix
• fix_undef_var_strict_warning
• igrishnov/gh-185-fix-version-file-missing
• igrishnov/gh-204-add-hardcoded-versioning-support
• ligurio/add-code-coverage-support
• ligurio/gh-137-connections
• ligurio/gh-76-fix-http_server-stop
• ligurio/shuffling-tests
• master
• oleg-jukovec/release-1.6.0
• psergee/gh-143-1.x.x-changelog
• refs/pull/173/merge
• refs/pull/174/merge
• refs/pull/177/merge
• refs/pull/178/merge
• refs/pull/179/merge
• refs/pull/180/merge
• refs/pull/181/merge
• refs/pull/183/merge
• refs/pull/184/merge
• refs/pull/186/merge
• refs/pull/187/merge
• refs/pull/188/merge
• refs/pull/190/merge
• refs/pull/192/merge
• refs/pull/194/merge
• refs/pull/197/merge
• refs/pull/198/merge
• refs/pull/200/merge
• refs/pull/201/merge
• refs/pull/203/merge
• refs/pull/204/merge
• refs/pull/206/merge
• refs/pull/208/merge
• refs/pull/210/merge
• refs/pull/211/merge
• refs/pull/213/merge
• refs/pull/214/merge
• refs/pull/215/merge
• refs/pull/218/merge
• refs/pull/220/merge
• refs/pull/221/merge
• refs/pull/222/merge
• refs/tags/1.3.0
• refs/tags/1.4.0
• refs/tags/1.5.0
• refs/tags/1.6.0
• refs/tags/1.7.0
• refs/tags/1.8.0
• themilchenko/add-sslsocket-file-into-rockspec
• themilchenko/fix-sslsocket-support
• themilchenko/get-rid-of-deleted-job
• themilchenko/gh-199-add-ssl-support-httpd-role
• themilchenko/gh-209-fix-server-address-change
• themilchenko/gh-216-fix-update-opts-on-config-reload
• themilchenko/gh-35-add-ssl-support
• themilchenko/release-1.7.0
• themilchenko/release-1.8.0
• themilchenko/support-mtls

Build # 15491049037

Build Type

Pull #210

github

Committed by olegrok

Commit Message

sslsocket: fix "attempt to redefine 'SSL_*'" error

Different libraries can use FFI OpenSSL bindings.
E.g. cartridge already uses it:
https://github.com/tarantool/cartridge/blob/master/cartridge/sslsocket.lua

Right now there is no any significant difference in cdef. But in
future it can be different (e.g. after implementation of #207).
Since we use "struct SSL_METHOD {} SSL_METHOD" definition
it can cause errors like 'attempt to
redefine 'SSL_METHOD'" that lead to the case then pcall silently
handle an error but some symbols won't be defined.

This patch fixes such case.
That's only relevant for struct definitions. Functions and typedefs can be redefined.

Pull Request Pull Request #210: sslsocket: fix "attempt to redefine 'SSL_*'" error

Run Details

5 of 5 new or added lines in 1 file covered. (100.0%)

867 of 1101 relevant lines covered (78.75%)

72.7 hits per line