engine icon indicating copy to clipboard operation
engine copied to clipboard

валидация CRYPT_PARAMS

Open chipitsine opened this issue 3 years ago • 7 comments

баг репорт от @Rostislaved

при не очень аккуратной сборке образа докер, в конфиг доехало вот такое

CRYPT_PARAMS = id-Gost28147-89-CryptoPro-A-ParamSet RUN cd /usr/local/src

gost-engine-photo_2022-02-08_12-22-28

в итоге - ожидаемым образом не работало.

мысль - можно ли в случае невалидного конфига делать hard fail ? в идеале с ошибкой "валидация конфига не прошла"

chipitsine avatar Feb 08 '22 07:02 chipitsine

Во-первых, не надо пихать в конфиг устаревшие параметры :) Во-вторых, чтобы проверить, что тут у нас фигня написана, надо явно это проверять. А так - строка и строка. В-главных, проблема в том, что в openssl нет механизма warning, чтобы, увидев здесь фигню, осуществить fallback к значению по умолчанию и дать warning.

beldmit avatar Feb 08 '22 08:02 beldmit

жила-была девочка. сама виновата

chipitsine avatar Feb 08 '22 08:02 chipitsine

мысль примерно такая - эти параметры обрабатываются в openssl или он их делегирует в engine ? если в engine, можно было бы там и фейлить

chipitsine avatar Feb 08 '22 08:02 chipitsine

ну не без того. Вообще это значение параметров шифрования я из рекомендаций выкинул достаточно давно, и добавил ошибку - но не warning. Но народ упорно копирует из старых инструкций - это умолчание было релевантно лет 10 назад.

делегирует в engine, вот это место: https://github.com/gost-engine/engine/blob/374627ab520fca0a189d56a747abbff7e2b67bd7/gost_crypt.c#L326-L341

beldmit avatar Feb 08 '22 08:02 beldmit

https://github.com/gost-engine/engine/blob/master/example.conf - вот отсюда вполне могут копировать.

chipitsine avatar Feb 08 '22 13:02 chipitsine

а что предполагается должно произойти после ERR_add_error_data на стороне openssl ? выглядит, как будто должно упасть. баг ?

chipitsine avatar Feb 08 '22 13:02 chipitsine

По идее неправильное значение должно попасть в лог, и дальше можно прогрепать. Ща уберу значение из example.conf, спасибо!

beldmit avatar Feb 08 '22 13:02 beldmit