35305 2018-08-29 13:28:36 +0300 Qt5Network динамически подгружает случайные библиотеки libssl/libcrypto 2018-08-30 11:31:26 +0300 1 1 4 Development Sisyphus libqt5-network unstable all Linux CLOSED FIXED P3 blocker --- 35303 1 lav zerg aen evg glebfm ldv zerg qa-sisyphus oldest_to_newest 173682 0 lav 2018-08-29 13:28:36 +0300 Возможно, это не совсем так, но судя по бинарнику, Qt5Network динамически подгружает любую библиотеку libcrypto, которую найдёт: $ strings /usr/lib64/libQt5Network.so.5.11.1 | grep libssl libssl.* $ strings /usr/lib64/libQt5Network.so.5.11.1 | grep libcrypto libcrypto.* libqt5-network-5.11.1-alt2.S1.x86_64 173683 1 lav 2018-08-29 13:33:06 +0300 При запуске программы получаем в консоли qt.network.ssl: QSslSocket: cannot resolve SSLv23_client_method qt.network.ssl: QSslSocket: cannot resolve SSLv2_server_method qt.network.ssl: QSslSocket: cannot resolve SSLv3_server_method qt.network.ssl: QSslSocket: cannot resolve SSLv23_server_method qt.network.ssl: QSslSocket: cannot resolve X509_STORE_CTX_get_chain qt.network.ssl: QSslSocket: cannot resolve OPENSSL_add_all_algorithms_noconf qt.network.ssl: QSslSocket: cannot resolve OPENSSL_add_all_algorithms_conf qt.network.ssl: QSslSocket: cannot resolve SSLeay qt.network.ssl: Incompatible version of OpenSSL 173685 2 zerg 2018-08-29 14:36:59 +0300 Как вариант: имена файлов подгружаемых библиотек сделаны так, что только у нас такое поведение. 173686 3 glebfm 2018-08-29 14:40:02 +0300 (In reply to comment #2) > Как вариант: имена файлов подгружаемых библиотек сделаны так, что только у нас > такое поведение. Ну, у 1.0 у нас, вроде, было другое название. А у 1.1 у нас такое же как в апстриме. А что там за логика? 173687 4 zerg 2018-08-29 14:41:31 +0300 Например [zerg@zerg ~]$ rpmvercmp libcrypto.so.10 libcrypto.so.1.1 1 [zerg@zerg ~]$ rpmvercmp libcrypto.so.10 libcrypto.so.1.0.2p 1 173688 5 zerg 2018-08-29 14:42:57 +0300 [zerg@zerg ~]$ rpmvercmp libcrypto.so.10 libcrypto.so.0 1 173689 6 glebfm 2018-08-29 14:52:23 +0300 (In reply to comment #5) > [zerg@zerg ~]$ rpmvercmp libcrypto.so.10 libcrypto.so.0 > 1 qt использует rpmvercmp для выбора libcrypto? 173690 7 zerg 2018-08-29 14:56:07 +0300 "Вполне возможно". Надо посмотреть. 173691 8 zerg 2018-08-29 15:02:05 +0300 (В ответ на комментарий №6) > qt использует rpmvercmp для выбора libcrypto? Видимо, да. Используется функтор LibGreaterThan из http://git.altlinux.org/people/zerg/packages/qt5-base.git?p=qt5-base.git;a=blob;f=src/network/ssl/qsslsocket_openssl_symbols.cpp 173692 9 zerg 2018-08-29 15:04:44 +0300 (В ответ на комментарий №8) > Используется функтор LibGreaterThan через NumericallyLess в том же файле 173693 10 glebfm 2018-08-29 15:06:26 +0300 А нельзя просто слинковать библиотеку с openssl? 173694 11 zerg 2018-08-29 15:14:22 +0300 (В ответ на комментарий №10) > А нельзя просто слинковать библиотеку с openssl? Можно, но я уже отказался от этой затеи. 173695 12 glebfm 2018-08-29 15:16:58 +0300 (In reply to comment #11) > (В ответ на комментарий №10) > > А нельзя просто слинковать библиотеку с openssl? > Можно, но я уже отказался от этой затеи. Расскажешь почему? 173697 13 zerg 2018-08-29 15:23:21 +0300 (В ответ на комментарий №12) > Расскажешь почему? Чтобы зря не пересобирать. 173698 14 glebfm 2018-08-29 15:28:41 +0300 (In reply to comment #13) > (В ответ на комментарий №12) > > Расскажешь почему? > Чтобы зря не пересобирать. Один раз в openssl ABI? Думаешь, отказ от одной пересборки раз в много лет стоит тех проблем, которые мы имеем сейчас? 173699 15 zerg 2018-08-29 15:36:45 +0300 (В ответ на комментарий №14) > Один раз в openssl ABI? Думаешь, отказ от одной пересборки раз в много лет > стоит тех проблем, которые мы имеем сейчас? Не стоит, конечно. Пересоберите openssl. Достаточно добавить симлинки libcrypto.so.11 -> libcrypto.so.1.1 libcrypto.so.11 -> libcrypto.so.1.1 173700 16 zerg 2018-08-29 15:37:29 +0300 (В ответ на комментарий №15) > Достаточно добавить симлинки > libcrypto.so.11 -> libcrypto.so.1.1 и libssl.so.11 -> libssl.so.1.1 173701 17 glebfm 2018-08-29 15:48:29 +0300 (In reply to comment #15) > (В ответ на комментарий №14) > > Один раз в openssl ABI? Думаешь, отказ от одной пересборки раз в много лет > > стоит тех проблем, которые мы имеем сейчас? > Не стоит, конечно. Пересоберите openssl. > Достаточно добавить симлинки > libcrypto.so.11 -> libcrypto.so.1.1 > libcrypto.so.11 -> libcrypto.so.1.1 Объясни, пожалуйста, чем этот хак лучше, чем просто честно собрать qt5 с библиотекой, которую он использует? Очевидно, что дополнительная пересборка один раз в новый openssl ABI не может перевесить преимущества линковки. 173702 18 zerg 2018-08-29 16:03:51 +0300 (В ответ на комментарий №17) > Объясни, пожалуйста, чем этот хак лучше, чем просто честно собрать qt5 с > библиотекой, которую он использует? Потому, что openssl сломал, его и чинить. > Очевидно, что дополнительная пересборка один раз Не один раз, а каждый раз. И у нас не один Qt. 173703 19 aen 2018-08-29 16:08:16 +0300 (В ответ на комментарий №18) > (В ответ на комментарий №17) > > Объясни, пожалуйста, чем этот хак лучше, чем просто честно собрать qt5 с > > библиотекой, которую он использует? > Потому, что openssl сломал, его и чинить. > Сергей, но openssl не сломан. 173704 20 glebfm 2018-08-29 16:09:53 +0300 (In reply to comment #18) > (В ответ на комментарий №17) > > Объясни, пожалуйста, чем этот хак лучше, чем просто честно собрать qt5 с > > библиотекой, которую он использует? > Потому, что openssl сломал, его и чинить. > > > Очевидно, что дополнительная пересборка один раз > Не один раз, а каждый раз. И у нас не один Qt. 173705 21 glebfm 2018-08-29 16:13:40 +0300 (In reply to comment #18) > (В ответ на комментарий №17) > > Объясни, пожалуйста, чем этот хак лучше, чем просто честно собрать qt5 с > > библиотекой, которую он использует? > Потому, что openssl сломал, его и чинить. openssl обновился и показал нам багу в qt. > > Очевидно, что дополнительная пересборка один раз > Не один раз, а каждый раз. И у нас не один Qt. Раз в 8 лет? У qt есть зависимости которые обновляются гораздо чаще. 173706 22 zerg 2018-08-29 16:20:25 +0300 (В ответ на комментарий №21) > openssl обновился и показал нам багу в qt. Если криво обновляться, кого угодно можно сломать. (В ответ на комментарий №19) > Сергей, но openssl не сломан. Это как раз Qt не сломан. Т.к. openssl "починился" с его soname-ом, то ему и разгребать. 173708 23 aen 2018-08-29 16:25:29 +0300 Сергей, openssl библиотека более низкого уровня. И мы пересобираем другие пакеты под изменения в ней. Пожалуйста, не надо спорить попусту. Изменения были анонсированы и надо было обсуждать их раньше. А сейчас уже идёт пересборка. 173709 24 aen 2018-08-29 16:29:30 +0300 Ладно, давайте спросим мнение главного конструктора. 173710 25 glebfm 2018-08-29 16:31:09 +0300 qt5 в Сизифе собран таким образом, чтобы нельзя было отследить его разлом до того как этот разлом произойдёт. Я считаю, что это ошибка сборки, которую следует исправить. openssl только позволил увидеть эту проблему, в остальном он совершенно не при чём. 173711 26 zerg 2018-08-29 16:33:13 +0300 Текущая схема ДО СИХ ПОР не ломалась. И я предложил простое решение. В будущем можно его легко выкинуть. 173712 27 zerg 2018-08-29 16:36:10 +0300 (В ответ на комментарий №23) > Изменения были анонсированы Изменения наименования soname анонсировано не было. 173713 28 glebfm 2018-08-29 16:38:07 +0300 И всё же, почему нельзя просто слинковать qt5 с openssl? Ответь, пожалуйста, на этот вопрос. 173714 29 ldv 2018-08-29 16:38:48 +0300 qt5 ОБЯЗАНА быть слинкована с теми libcrypto/libssl, которые использует, чтобы все статические проверки ABI, которые у нас реализованы, работали, и пользователи не страдали от таких ошибок, как эта. Сергей, исправь qt5 НЕЗАМЕДЛИТЕЛЬНО! 173715 30 zerg 2018-08-29 17:11:19 +0300 (В ответ на комментарий №28) > И всё же, почему нельзя просто слинковать qt5 с openssl? Кто сказал "нельзя"? Я считаю более удобным не линковать. 173716 31 zerg 2018-08-29 17:16:02 +0300 (В ответ на комментарий №29) > qt5 ОБЯЗАНА быть слинкована с теми libcrypto/libssl, которые использует, > чтобы все статические проверки ABI, которые у нас реализованы, работали Ну, хоть аргумент появился. >, и пользователи не страдали от таких ошибок, как эта. От таких больше не будут, только во всех местах, где "не как у всех", вылазили, вылазят и будут вылазить другие. > Сергей, исправь qt5 НЕЗАМЕДЛИТЕЛЬНО! Ok. 173718 32 zerg 2018-08-29 18:36:55 +0300 http://webery.altlinux.org/task/212070 173722 33 aen 2018-08-30 11:31:26 +0300 Сергей, спасибо!