LINUX.ORG.RU
Форум — Talks  

goto fail

 , ,


0

1

https://www.imperialviolet.org/2014/02/22/applebug.html

Ъ:

static OSStatus
SSLVerifySignedServerKeyExchange(SSLContext *ctx, bool isRsa, SSLBuffer signedParams,
                                 uint8_t *signature, UInt16 signatureLen)
{
	OSStatus        err;
	...

	if ((err = SSLHashSHA1.update(&hashCtx, &serverRandom)) != 0)
		goto fail;
	if ((err = SSLHashSHA1.update(&hashCtx, &signedParams)) != 0)
		goto fail;
		goto fail;
	if ((err = SSLHashSHA1.final(&hashCtx, &hashOut)) != 0)
		goto fail;
	...

fail:
	SSLFreeBuffer(&signedHashes);
	SSLFreeBuffer(&hashCtx);
	return err;
}
ЛОРовец, а ты тоже пишешь if без скобок, как последний содомит?

★★★★★

Последнее исправление: Suntechnic (всего исправлений: 2)

 ← 1 2 →

The first one is correctly bound to the if statement but the second, despite the indentation, isn't conditional at all.

кто там инденты в питоне чмырит?

MyTrooName ★★★★★
()

они там на своей гомояхте не в курсе про тестирование кода?

MyTrooName ★★★★★
()
Ответ на: комментарий от unt1tled

Меня бесят такие ифы. Но хуже всего if(cond): code endif; Они бывают огромны и при этом endif не подсвечивается редактором, сцуко.

Suntechnic ★★★★★
() автор топика
Ответ на: комментарий от Suntechnic

даа, сам не люблю конструкций типа 

if(1 == 1) doStuff();

всмысле в одну строку когда написано
а лишние скобки тупо бесят

unt1tled ★★★★
()
Последнее исправление: unt1tled (всего исправлений: 1)
Ответ на: комментарий от Suntechnic

в питоне блок кода опреляется не скобками, а сдвигом от левого края:

if True:
   x = 1 # this is one...
   y = 2 # ...code block
 z = 4 # this is a syntax error
z = 3 # this is outside of `if`
MyTrooName ★★★★★
()
Последнее исправление: MyTrooName (всего исправлений: 1)

ЛОРовец, а ты тоже пишешь if без скобок

Да.

как последний содомит?

Не знаю, как пишут содомиты.

KRoN73 ★★★★★
()

Да. И еще циклы.
(По поводу бага: бугога!)

thesis ★★★★★
()
Ответ на: комментарий от KRoN73

Отлично от: 

if (cond) {
    code;
}

P.S. На самом деле просто надо было сопроводить копипасту каким-нибудь комментарием от себя ;)

Suntechnic ★★★★★
() автор топика
Последнее исправление: Suntechnic (всего исправлений: 1)
Ответ на: комментарий от Suntechnic

я так косячил дважды на моей памяти: в первый раз когда еще питона до этого в глаза не видел.

заметить такой баг имхо сложнее, чем s/=/==. последний хотя б варнингами детектится

MyTrooName ★★★★★
()
Ответ на: комментарий от Suntechnic

да в shell-скрипте команда rm -fr / usr/чего-то-там-для-невидии
багтрекер трещал от злых юзверей, у который / заделан под нуль =D

ubuntuawp ★★
()
Ответ на: комментарий от Suntechnic

В скрипте установки бамблби в пути для rm затесался пробел между /usr и остальной частью пути.

PolarFox ★★★★★
()
Ответ на: комментарий от Suntechnic

Отлично от

Ну, не одобряя содомизм как таковой, не могу за содомитов не порадоваться, что они такой ужас не пишут :)

KRoN73 ★★★★★
()

В последней мацоси есть еще и такая бяка: отправляешь ноут спать, просыпаешь обратно, опять спать, опять просыпаешь, и внезапно обнаруживаешь, что у тебя стек TCP/IP по косому женскому гениталию пошел: внезапно, дескрипторы все закончились. Симптомы настолько же странные, как и внезапно закончившееся место на диске, только в контексте сети.

Это было в 10.9, в 10.9.1 не починили.

Самое смешное, что этот баг на хакинтоше не работает, потому что там не работает режим сна. :)

shimon ★★★★★
()

ЛОРовец, а ты тоже пишешь if без скобок,

Да

qrck ★★
()
Ответ на: комментарий от KRoN73

Ну, я привел в пример код настоящего содомита, Эрика Оллмана.

X10Dead ★★★★★
()
Последнее исправление: X10Dead (всего исправлений: 1)
Ответ на: комментарий от anonymous_sapiens

то ли дело у нас приколы после ведроапдейтов - отправил ноут спать - не проснулся.

Так то отродясь. Что-то работает или ценой неимоверных танцев с убубном получилось — вау, праздник.

shimon ★★★★★
()

ЛОРовец, а ты тоже пишешь if без скобок

Да.

как последний содомит?

Нет.

tailgunner ★★★★★
()

Я всё починил, куда слать патч? 

if ((err = SSLHashSHA1.update(&hashCtx, &serverRandom)) != 0 || (err = SSLHashSHA1.update(&hashCtx, &signedParams)) != 0 || (err = SSLHashSHA1.final(&hashCtx, &hashOut)) != 0) {
	SSLFreeBuffer(&signedHashes);
	SSLFreeBuffer(&hashCtx);
	return err;
}

Lavos ★★★★★
()
Последнее исправление: Lavos (всего исправлений: 1)

ох уж эти эстеты... не нравится, пиши сам! :)

der_looser ★★
()
Вы не можете добавлять комментарии в эту тему. Тема перемещена в архив.
 ← 1 2 →
Talks