The OpenNET Project / Index page

[ новости /+++ | форум | теги | ]



Вариант для распечатки  
Пред. тема | След. тема 
Форум Разговоры, обсуждение новостей
Режим отображения отдельной подветви беседы [ Отслеживать ]

Оглавление

Спорная ошибка в systemd, позволяющая повысить привилегии, з..., opennews (ok), 02-Июл-17, (0) [смотреть все]

Сообщения [Сортировка по времени | RSS]


46. "Спорная ошибка в systemd, позволяющая повысить привилегии, з..."  +10 +/
Сообщение от Аноним (-), 03-Июл-17, 00:59 
Ну, во-первых нет такого типа, чтобы сам по себе имел диапазон 1-100. uint8_t - это [0,256]. Во-вторых, для целых чисел недостаточной разрядности есть опасность переполнения типа. В-третьих, пётеринг отказывается делать проверку, мотивируя тем, что "вам запрещено передавать в функцию числа больше 100". То что по факту хакеры вертели его запреты на детородном органе, сабж признавать отказывается.

Так что да, тут злостное нарушение практики defensive programming в критически важном компоненте системы, и отказ признать свою неправоту после прицельного тычка носом в получившуюся кучу.

Ответить | Правка | К родителю #37 | Наверх | Cообщить модератору

50. "Спорная ошибка в systemd, позволяющая повысить привилегии, з..."  –11 +/
Сообщение от Аноним (-), 03-Июл-17, 01:40 
Ещё раз: есть здравый смысл (на который ты напираешь), а есть ТЗ. Если в ТЗ написано что будет, значит это БУДЕТ. Как, почему, а может всё-таки стоит - это неправильные вопросы. Если в ТЗ написано что пункт меню должен называться "Параматеры", то он ДОЛЖЕН называться "Параматеры". Это требование (ТРЕБОВАНИЕ!!!!!) и ты обязан (ОБЯЗАН) его выполнить.

Это как описание API: не надо додумывать и переиначивать то, что написано прямым текстом.

Напр. функция поиска подстроки в php должна вернуть "Возвращает указанную подстроку. Если подстрока не найдена, возвращается FALSE". Не может быть числа на выходе. Не может быть массива на выходе. Не может и всё тут. Но ты ведь всё равно проверишь что это не число и не массив, да?

Ответить | Правка | Наверх | Cообщить модератору

55. "Спорная ошибка в systemd, позволяющая повысить привилегии, з..."  +13 +/
Сообщение от Crazy Alex (ok), 03-Июл-17, 01:52 
Чувак, иди учись дальше. Входные данные, за которые ты не можешь поручиться головой (то есть пришедшие откуда угодно извне) всегда проверяются. Это основа основ.

P.S. А входные данные, за которые ты головой поручиться таки готов - лучше тоже проверить, если ресурсы позволяют. Потому что от ошибок в своём коде никто не застрахован.

Ответить | Правка | Наверх | Cообщить модератору

75. "Спорная ошибка в systemd, позволяющая повысить привилегии, з..."  –9 +/
Сообщение от Аноним (-), 03-Июл-17, 04:40 
Если проверка параметров нужна (а обычно она нужна), то делается функция-обёртка, которая только и делает что проверяет параметры и вызывает функцию-обработчик. Обёртка может выполнять кэширование, проверку прав доступа и многое другое.

Кто будет писать обёртку тебя не касается, это не ты.

А того, кто будет писать обёртку, не касается кем, где и как будет реализован функционал.

Есть паттерны, которые называются Заместитель и Декоратор, но ты с ними явно не знаком. И к проектной работе ты тоже не готов.

Ответить | Правка | Наверх | Cообщить модератору

77. "Спорная ошибка в systemd, позволяющая повысить привилегии, з..."  –7 +/
Сообщение от Аноним (-), 03-Июл-17, 04:53 
У царя Мидаса всё в золото превращалось, а у вас - в дерьмо. Вечно хотите запихать в функцию и проверку параметров, и функционал, и отрисовку, и кэширование.

Подходишь, ничего не сделано. "А я тут занимался ***". Ну вот на *** ты этим занимался?????!!!!! Тебя просили? Тебе прямым текстом сказали "не надо". Ты дурак???? "Но ведь это тоже надо делать..." Надо. Но не тебе. И не так.

Ответить | Правка | К родителю #55 | Наверх | Cообщить модератору

78. "Спорная ошибка в systemd, позволяющая повысить привилегии, з..."  +9 +/
Сообщение от Аноним (-), 03-Июл-17, 05:16 
> У царя Мидаса всё в золото превращалось, а у вас - в
> дерьмо. Вечно хотите запихать в функцию и проверку параметров, и функционал,
> и отрисовку, и кэширование.
> Подходишь, ничего не сделано. "А я тут занимался ***". Ну вот на
> *** ты этим занимался?????!!!!! Тебя просили? Тебе прямым текстом сказали "не
> надо". Ты дурак???? "Но ведь это тоже надо делать..." Надо. Но
> не тебе. И не так.

Подходишь такой к Поттерингу, попросил показать инит. "А я сделал комбайн. Тут DNS-сервер, сканер QR-кодов, бинарные логи, и ещё 20 ненужных вещей"

Ответить | Правка | Наверх | Cообщить модератору

162. "Спорная ошибка в systemd, позволяющая повысить привилегии, з..."  +/
Сообщение от angra (ok), 03-Июл-17, 18:10 
Подходишь, ничего не сделано. "А я тут занимался проектированием с применением десятка патернов, о которых на днях прочитал в умной книжке. Ну вот на *** ты этим занимался?????!!!!! Тебя просили? Тебе прямым текстом сказали "не надо". Ты дурак????"
Ответить | Правка | К родителю #77 | Наверх | Cообщить модератору

117. "Спорная ошибка в systemd, позволяющая повысить привилегии, з..."  +1 +/
Сообщение от anon3000 (?), 03-Июл-17, 11:06 
Ничо, ничо. Начнёшь проверять все данные сразу после первого серьёзного факапа из-за некорректных данных.
Ответить | Правка | К родителю #50 | Наверх | Cообщить модератору

164. "Спорная ошибка в systemd, позволяющая повысить привилегии, з..."  +/
Сообщение от angra (ok), 03-Июл-17, 18:16 
Не начнет, он же манагер по образу мышления. Отбрехается тем, что в ТЗ этого не было, а ТЗ согласовали, вот подписи. А то, что проверка данных ожидается по умолчанию от качественного кода, его не колышет, он без понятия об этих умолчаниях, у него бумажка с ТЗ есть.

Ответить | Правка | Наверх | Cообщить модератору

118. "Спорная ошибка в systemd, позволяющая повысить привилегии, з..."  +/
Сообщение от Аноним (-), 03-Июл-17, 11:21 
>Ещё раз: есть здравый смысл (на который ты напираешь), а есть ТЗ.

А где оно, ТЗ на системду, не подскажешь?

Ответить | Правка | К родителю #50 | Наверх | Cообщить модератору

54. "Спорная ошибка в systemd, позволяющая повысить привилегии, з..."  +/
Сообщение от Crazy Alex (ok), 03-Июл-17, 01:50 
Вообще-то такие типы есть - в Аде, например (хм, init на Аде - интересная мысль, в принципе). В остальном - согласен.
Ответить | Правка | К родителю #46 | Наверх | Cообщить модератору

59. "Спорная ошибка в systemd, позволяющая повысить привилегии, з..."  +/
Сообщение от Аноним (-), 03-Июл-17, 02:20 
> Вообще-то такие типы есть - в Аде, например

Паскальщина же, ну и заодно модула.

Ответить | Правка | Наверх | Cообщить модератору

173. "Спорная ошибка в systemd, позволяющая повысить привилегии, з..."  –1 +/
Сообщение от Аноним (-), 03-Июл-17, 19:33 
Модулу не знаю, а на паскалях (FP и Delphi) это делается только через аналог enum. Внутри там те же integer-ы с разрядностью как у процессора. Более того, рантайм может не отлавливать получение числа, не попадающего в разрешённый диапазон.

То есть проверять надо.

Ответить | Правка | Наверх | Cообщить модератору

184. "Спорная ошибка в systemd, позволяющая повысить привилегии, з..."  +/
Сообщение от Led (ok), 04-Июл-17, 01:19 
> Модулу не знаю, а на паскалях (FP и Delphi) это делается только через аналог enum.

Садись - "двойка"!

var I:1..100;

> Внутри там те же integer-ы с разрядностью как
> у процессора. Более того, рантайм может не отлавливать получение числа, не
> попадающего в разрешённый диапазон.

А может и "отлавливать". Начиная с... врать не буду, но начиная с TurboPascal 5.0 for DOS - точно мог "отлавливать".

> То есть проверять надо.

Т.е. уроки прогуливать не надо.

Ответить | Правка | Наверх | Cообщить модератору

108. "Спорная ошибка в systemd, позволяющая повысить привилегии, з..."  +/
Сообщение от Аноним (-), 03-Июл-17, 10:35 
>нет такого типа, чтобы сам по себе имел диапазон 1-100

ты не умеешь в типизацию

Ответить | Правка | К родителю #46 | Наверх | Cообщить модератору

130. "Спорная ошибка в systemd, позволяющая повысить привилегии, з..."  –1 +/
Сообщение от Аноним (-), 03-Июл-17, 12:22 
enum?
Ответить | Правка | К родителю #46 | Наверх | Cообщить модератору

215. "Спорная ошибка в systemd, позволяющая повысить привилегии, з..."  +/
Сообщение от Pilat (ok), 14-Июл-17, 14:43 
> Ну, во-первых нет такого типа, чтобы сам по себе имел диапазон 1-100.
> uint8_t - это [0,256].

Скоро кто-то прочитает Ваш пост и поверит что это именно так. И сделает очередную чудо-приблуду, с которой Поттерингу придётся разбираться.

Ответить | Правка | К родителю #46 | Наверх | Cообщить модератору

Архив | Удалить

Рекомендовать для помещения в FAQ | Индекс форумов | Темы | Пред. тема | След. тема




Партнёры:
PostgresPro
Inferno Solutions
Hosting by Hoster.ru
Хостинг:

Закладки на сайте
Проследить за страницей
Created 1996-2024 by Maxim Chirkov
Добавить, Поддержать, Вебмастеру