> Ну в общем классический креационист. Ссылки на каких-то там "все большее число
> ученых отказываются от эволюционизма".Да нате, мне не жалко:
https://www.opennet.ru/openforum/vsluhforumID3/110746.html#21
...
Пункт первый - качество кода/архитектуры:
Перлы в коде, типа магических строк и прочего:
arg_header = strdup(option+7);
strcpy(stpcpy(stpcpy(stpcpy(mempcpy(t, p, fn - p), ".#"), extra), fn), «XXXXXX»);
if (path_is_absolute(option+15))
ret = new(char, (e - slice) + 1 + strlen(name) + 6 + 1);
strcpy(mempcpy(mempcpy(r, f, a + 1), i, b), e);
...
это еще не задаваясь вопросом, зачем вообще понадобилось изобретать велосипед вместо использования того же re2/lemonparser. Ну или хотя бы, если уж так зудит NIH-синдром, почитать классику и сделать сц*ко _нормальный_ токенизатор и парсер (благо, там грамматика простейшая и простого recursive descent парсера должно хватить за глаза) , а не велосипедить что-то с strchr/strspn, распихивая все это действо по всему коду.
...
там еще есть интересный пунктик с проверками входных данных - они довольно бессистемны и дубли-"триплируются".
...
второй пункт претензий - запихивание всего, чего можно:
https://www1.opennet.ru/opennews/art.shtml?num=41301
> В Systemd добавлен код для разбора формата JSON
> В дополнение к уже присутствующей поддержке формата XML
Про классику типа QR кодов и встроенного httpd скромно умолчим. Тот же ресольвер по фунциональности хромает на обе ноги, зато с дырами там все нормально https://www.openwall.com/lists/oss-security/2017/06/27/8
Третий:
https://lists.freedesktop.org/archives/systemd-devel/2016-Fe...
> * Most configurable timeouts in systemd now expect an argument of "infinity" to turn them off, instead of "0" as before.
А чтобы жизнь медом не казалась
> To maintain backwards compatibility, "0" continues to turn off previously existing timeout settings
или
https://lwn.net/Articles/490413/
> you can still build it for usage outside of systemd systems, and we will support these builds
> officially. In fact, we will be supporting this for a long time
Да, лонг-тайм оказался периодом аж в два года.
Или очередной велосипедизм с элементами гвоздеприбивания колес:
https://lists.freedesktop.org/archives/systemd-devel/2015-Fe...
> When the user presses Ctrl-Alt-Del more than 7x within 2s an immediate reboot is triggered.
>
Ну и "мытеперьвашновыйстандарт!":
https://github.com/systemd/systemd/issues/825#issuecomment-1...
>poettring:
> Long story short: "su" is really a broken concept. It will given you kind of a shell, and it's fine to use it for that, but it's not a full login, and shouldn't be mistaken for one.
https://github.com/systemd/systemd/issues/5644
> tmpfiles: R! /dir/.* destroys root
-
> poettering commented Mar 30, 2017
> I am not sure I'd consider this much of a problem. Yeah, it's a UNIX
> pitfall, but "rm -rf /foo/.*" will work the exact same way, no?
>
===========
Еще конкретики? Их есть у нас:
https://www.opennet.ru/openforum/vsluhforumID3/110746.html#82
https://github.com/systemd/systemd/blob/master/src/basic/mac...
> На 238 пушка просто: /* We override the glibc assert() here. */. Молодец, Леня!
> В макросе на 323 ошибка: не учитывается знак минуса для отрицательных чисел (а они, судя по комментарию
> для соседнего макроса на стр. 313-316, ожидаются).
> Опасный макрос на 333.
> Какой-то ад на строках 335-361.
>
