11090 2007-03-14 12:06:54 +0300 Непонятное описание в помощи к passkey-agent, есть ошибка в патче. 2008-10-06 19:48:48 +0400 1 1 4 Development Sisyphus bluez-utils unstable all Linux CLOSED NOTABUG P4 major --- 1 serpiph wrar genix ktirf mike mithraen shrek week wrar qa-sisyphus oldest_to_newest 46603 0 serpiph 2007-03-14 12:06:54 +0300 в 'usage' от passkey-agent написаны только длинные параметры ('--default' и '--path'), но что есть и короткие ('-d' и '-p'), можно узнать только или из интернета, или из исходников самого passkey-agent. Как и требование, что ОБЯЗАТЕЛЬНО должен быть или ключ '-d', или адрес подключения к D-Bus. Да и про '--path' я только из исходников узнал, что это адрес программы в D-Bus, а не на диске. Ещё нет информации про '--help' или '-h'. Плюс в коде патча bluez-utils-3.9-alt-pin-exec.patch опечатка: вместо ================= + if(buffer[1]!='P' && buffer[1]!='I' && buffer[2]!='N') + { + free(buffer); + return DBUS_HANDLER_RESULT_NOT_YET_HANDLED; + } =================== должно быть как минимум ================= + if(buffer[0]!='P' || buffer[1]!='I' || buffer[2]!='N' || buffer[3]!=':') + { + free(buffer); + return DBUS_HANDLER_RESULT_NOT_YET_HANDLED; + } =================== Нет НИКАКОГО указания, что строка, получаемая от helper'а, ДОЛЖНА быть в виде PIN:1234 БЕЗ пробела между ":" и самим кодом. 46605 1 serpiph 2007-03-14 12:17:42 +0300 *** Bug 11089 has been marked as a duplicate of this bug. *** 46656 2 wrar 2007-03-14 22:40:54 +0300 Присылайте патчи. Смею напомнить, что passkey-agent для упаковки вообще не предназначен. 46682 3 serpiph 2007-03-15 12:39:28 +0300 (In reply to comment #2) > Присылайте патчи. Попробую сделать. Есть вопрос: патч к данному патчу делать или к исходному тексту? В самом патче требуемые изменения я уже прописал. > Смею напомнить, что passkey-agent для упаковки вообще не предназначен. Предназначен-не предназначен - это другой вопрос. passkey-agent реально запакован? Тогда необходимо исправить ошибки. Если уж ничего не получится и Вы выкинете passkey-agent из пакета, то и баг тоже, фактически, уйдёт. 46705 4 wrar 2007-03-15 20:17:55 +0300 (In reply to comment #3) > > Присылайте патчи. > Попробую сделать. Есть вопрос: патч к данному патчу делать или к исходному > тексту? В самом патче требуемые изменения я уже прописал. К исходникам. > > Смею напомнить, что passkey-agent для упаковки вообще не предназначен. > Если уж ничего не получится и Вы выкинете passkey-agent из пакета Я не собираюсь его выкидывать за то, что к нему нет документации. 50096 5 wrar 2007-05-11 00:57:37 +0400 Патчей на документацию не вижу. Откуда инфа про двоеточие - не понимаю. Оно всё равно игнорируется. Описывать, в каком там формате строка - вообще не задача документации к passkey- agent, т.к. он в данном виде представляет собой обёртку к старым хелперам. 50101 6 serpiph 2007-05-11 11:23:03 +0400 Открываю заново. Насчёт двоеточия: все доступные мне файлы документации в интернете и в нашей рассылке все приводят пример такого предоставителя в виде: echo "PIN:1234" Свой патч сейчас приложу. 50102 7 1954 serpiph 2007-05-11 11:24:57 +0400 Created attachment 1954 Откорректирорванный патч из bluez-utils-3.9-alt2 Данный патч является откорректированным патчем bluez-utils-3.9-alt-pin-exec.patch из bluez-utils-3.9-alt2. 50129 8 wrar 2007-05-12 15:11:18 +0400 (In reply to comment #6) > Насчёт двоеточия: все доступные мне файлы документации в > интернете и в нашей рассылке все приводят пример такого предоставителя в виде: > echo "PIN:1234" Так а всё же, нахрена проверять это двоеточие? 50177 9 serpiph 2007-05-14 11:28:26 +0400 Если хотите, то проверку двоеточия можно удалить из патча. Но, на мой взгляд, такой пустой символ будет просто всех путать, так как всё равно passkey-agent начинает втихую брать с четвёртого символа. А пользователи могут написать: PIN1234 PIN 1234 PIN 1234 считая, что используемый пин всегда будет 1234. А на самом деле получим: 234 1234 1234 Может, лучше сказать жёстко: в виде выдавать "PIN:1234", а не просить писать его 1 (один) пробел после слова PIN? А то пойдут вопросы "а почему не два или три". Ну или писать интеллектуальный разборщик, который будет между словом "PIN" и кодовой строкой выбрасывать все пробелы. Но тогда как быть, если пользователь захочет в коде поставить пробел ВПЕРЕДИ? P.S. Кстати, в моем патче есть pclose() после чтения из буфера. Нужно ли оно? 50197 10 wrar 2007-05-14 19:03:43 +0400 (In reply to comment #9) > такой пустой символ будет просто всех путать, Какой пустой символ? > начинает втихую брать с четвёртого символа. А пользователи могут написать: > PIN1234 > PIN 1234 > PIN 1234 Пользователи с таким же успехом могут написать вообще что угодно. Вы же как-то догадались написать именно через двоеточие. > Может, лучше сказать жёстко: в виде выдавать "PIN:1234", а не просить писать его > 1 (один) пробел после слова PIN? А то пойдут вопросы "а почему не два или три". Сейчас вообще никто никому ничего не говорит. > P.S. Кстати, в моем патче есть pclose() после чтения из буфера. Нужно ли оно? Хотите сказать, что до вас его не было? 50227 11 serpiph 2007-05-15 12:58:33 +0400 (In reply to comment #10) > (In reply to comment #9) > > такой пустой символ будет просто всех путать, > Какой пустой символ? Четвёртый символ. Нигде нет информации, какой он должен быть и должен ли быть вообще. > > начинает втихую брать с четвёртого символа. А пользователи могут написать: > > PIN1234 > > PIN 1234 > > PIN 1234 > Пользователи с таким же успехом могут написать вообще что угодно. > Вы же как-то догадались написать именно через двоеточие. Только когда поднял документацию в интернете (которой, кстати раз-два и обчёлся). По исходнику совсем неясно. > > Может, лучше сказать жёстко: в виде выдавать "PIN:1234", а не просить писать > его > > 1 (один) пробел после слова PIN? А то пойдут вопросы "а почему не два или > три". > Сейчас вообще никто никому ничего не говорит. А зачем ждать, когда скажут? > > P.S. Кстати, в моем патче есть pclose() после чтения из буфера. Нужно ли оно? > Хотите сказать, что до вас его не было? Нет, если Вы сравните мой вариант и начальный, то увидите, что изначально pclose стоит ПОСЛЕ проверки условия на допустимость данных, а я сделал ДО. 50253 12 wrar 2007-05-15 19:42:05 +0400 (In reply to comment #11) > Четвёртый символ. Нигде нет информации, какой он должен быть и должен ли быть > вообще. А о первых трёх есть где-то информация? > > Пользователи с таким же успехом могут написать вообще что угодно. > Только когда поднял документацию в интернете (которой, кстати раз-два и > обчёлся). По исходнику совсем неясно. Вы думаете, что пользователи будут читать > > > Может, лучше сказать жёстко: в виде выдавать "PIN:1234", а не просить писать > > его > > > 1 (один) пробел после слова PIN? А то пойдут вопросы "а почему не два или > > три". > > Сейчас вообще никто никому ничего не говорит. > А зачем ждать, когда скажут? Вы запутались. > Нет, если Вы сравните мой вариант и начальный, то увидите, что изначально pclose > стоит ПОСЛЕ проверки условия на допустимость данных, а я сделал ДО. Т.е. он _был_. Хотя делать его до возможного return правильнее, да. 79248 13 wrar 2008-10-06 19:48:47 +0400 Author: Marcel Holtmann <marcel@holtmann.org> Date: Fri Oct 3 09:11:22 2008 +0200 Remove old passkey-agent and auth-agent 1954 2007-05-11 11:24:57 +0400 2007-05-11 11:24:57 +0400 Откорректирорванный патч из bluez-utils-3.9-alt2 bluez-utils-3.9-alt-pin-exec.patch text/plain 2547 serpiph ZGlmZiAtTmF1cnAgYmx1ZXotdXRpbHMtMy45Lm9yaWcvZGFlbW9uL3Bhc3NrZXktYWdlbnQuYyBi bHVlei11dGlscy0zLjkvZGFlbW9uL3Bhc3NrZXktYWdlbnQuYwotLS0gYmx1ZXotdXRpbHMtMy45 Lm9yaWcvZGFlbW9uL3Bhc3NrZXktYWdlbnQuYwkyMDA3LTAyLTE0IDA5OjMyOjQ0ICswMzAwCisr KyBibHVlei11dGlscy0zLjkvZGFlbW9uL3Bhc3NrZXktYWdlbnQuYwkyMDA3LTA0LTI2IDIyOjE1 OjMxICswNDAwCkBAIC03Nyw2ICs3NywxMSBAQCBzdGF0aWMgREJ1c0hhbmRsZXJSZXN1bHQgcmVx dWVzdF9tZXNzYWdlCiAJREJ1c01lc3NhZ2UgKnJlcGx5OwogCWNvbnN0IGNoYXIgKnBhdGgsICph ZGRyZXNzOwogCWRidXNfYm9vbF90IG51bWVyaWM7CisgCWNoYXIgKmJ1ZmZlcjsKKyAJRklMRSAq cGluOworIAl1bnNpZ25lZCBpbnQgYnVmbGVuOworIAlpbnQgc3RhdHVzOworIAljaGFyICpzX3Bh c3NrZXk7CiAKIAlpZiAoIXBhc3NrZXkpCiAJCXJldHVybiBEQlVTX0hBTkRMRVJfUkVTVUxUX05P VF9ZRVRfSEFORExFRDsKQEAgLTk1LDggKzEwMCwzNCBAQCBzdGF0aWMgREJ1c0hhbmRsZXJSZXN1 bHQgcmVxdWVzdF9tZXNzYWdlCiAJfQogCiAJcHJpbnRmKCJQYXNza2V5IHJlcXVlc3QgZm9yIGRl dmljZSAlc1xuIiwgYWRkcmVzcyk7CisgICAgYnVmbGVuPXN0cmxlbihwYXNza2V5KSsyNTY7Cisg ICAgaWYoYnVmbGVuPD0yNTYpCisgICAgeworICAgICAgICBmcHJpbnRmKHN0ZGVyciwgIkludmFs aWQgcGluLWhlbHBlciBhcmd1bWVudCwgYWJvcnRpbmcuIik7CisgICAgICAgIHJldHVybiBEQlVT X0hBTkRMRVJfUkVTVUxUX05PVF9ZRVRfSEFORExFRDsKKyAgICB9CisgICAgYnVmZmVyPW1hbGxv YyhidWZsZW4pOworICAgIGlmKGJ1ZmZlcj09TlVMTCkKKyAgICB7CisgICAgICAgIGZwcmludGYo c3RkZXJyLCAiRXJyb3IgYWxsb2NhdGluZyBtZW1vcnkgKG1hbGxvYyksIGFib3J0aW5nLiIpOwor ICAgICAgICByZXR1cm4gREJVU19IQU5ETEVSX1JFU1VMVF9OT1RfWUVUX0hBTkRMRUQ7CisgICAg fQorICAgIHNwcmludGYoYnVmZmVyLCAiJXMgJXMgJXMiLCBwYXNza2V5LCAiaW4iLCBhZGRyZXNz KTsKKyAgICBwaW49cG9wZW4oYnVmZmVyLCAiciIpOworICAgIHdhaXQoJnN0YXR1cyk7CisgICAg ZmdldHMoYnVmZmVyLCAyNTYsIHBpbik7CisgICAgcGNsb3NlKHBpbik7CisgICAgaWYoYnVmZmVy WzBdIT0nUCcgfHwgYnVmZmVyWzFdIT0nSScgfHwgYnVmZmVyWzJdIT0nTicgfHwgYnVmZmVyWzNd IT0nOicpCisgICAgeworICAgICAgZnJlZShidWZmZXIpOworICAgICAgcmV0dXJuIERCVVNfSEFO RExFUl9SRVNVTFRfTk9UX1lFVF9IQU5ETEVEOworICAgIH0KKyAgICBzX3Bhc3NrZXk9JmJ1ZmZl cls0XTsKKyAgICBpZihzX3Bhc3NrZXlbc3RybGVuKHNfcGFzc2tleSktMV09PSdcbicpIHNfcGFz c2tleVtzdHJsZW4oc19wYXNza2V5KS0xXT0nXDAnOworICAgIGZwcmludGYoc3RkZXJyLCAiRm91 bmQgcGFzc2tleTogXCIlc1wiXG4iLCBzX3Bhc3NrZXkpOwogCi0JZGJ1c19tZXNzYWdlX2FwcGVu ZF9hcmdzKHJlcGx5LCBEQlVTX1RZUEVfU1RSSU5HLCAmcGFzc2tleSwKKworCWRidXNfbWVzc2Fn ZV9hcHBlbmRfYXJncyhyZXBseSwgREJVU19UWVBFX1NUUklORywgJnNfcGFzc2tleSAsCiAJCQkJ CURCVVNfVFlQRV9JTlZBTElEKTsKIAogCWRidXNfY29ubmVjdGlvbl9zZW5kKGNvbm4sIHJlcGx5 LCBOVUxMKTsKQEAgLTEwNCw3ICsxMzUsNyBAQCBzdGF0aWMgREJ1c0hhbmRsZXJSZXN1bHQgcmVx dWVzdF9tZXNzYWdlCiAJZGJ1c19jb25uZWN0aW9uX2ZsdXNoKGNvbm4pOwogCiAJZGJ1c19tZXNz YWdlX3VucmVmKHJlcGx5KTsKLQorICAgICAgICBmcmVlKGJ1ZmZlcik7CiAJcmV0dXJuIERCVVNf SEFORExFUl9SRVNVTFRfSEFORExFRDsKIH0KIApAQCAtMzM0LDEwICszNjUsMTMgQEAgc3RhdGlj IGludCB1bnJlZ2lzdGVyX2FnZW50KERCdXNDb25uZWN0aQogCiBzdGF0aWMgdm9pZCB1c2FnZSh2 b2lkKQogewotCXByaW50ZigiQmx1ZXRvb3RoIHBhc3NrZXkgYWdlbnQgdmVyICVzXG5cbiIsIFZF UlNJT04pOworCXByaW50ZigiQmx1ZXRvb3RoIHBhc3NrZXkgYWdlbnQgdmVyICVzIHdpdGggXCJl eGVjXCIgcGF0Y2ggZnJvbSBNYXJjbyBHdWxpbm8gKGh0dHA6Ly93d3cua21vYmlsZXRvb2xzLm9y Zy9ub2RlLzIyOClcblxuIiwgVkVSU0lPTik7CiAKIAlwcmludGYoIlVzYWdlOlxuIgotCQkiXHRw YXNza2V5LWFnZW50IFstLWRlZmF1bHRdIFstLXBhdGggYWdlbnQtcGF0aF0gPHBhc3NrZXk+IFth ZGRyZXNzXVxuIgorCQkiXHRwYXNza2V5LWFnZW50IHstZHwtLWRlZmF1bHR9IFt7LXB8LS1wYXRo fSBhZ2VudC1wYXRoXSA8cGF0aC10by1waW4taGVscGVyPlxuIgorCQkiXHRwYXNza2V5LWFnZW50 IFt7LXB8LS1wYXRofSBhZ2VudC1wYXRoXSA8cGF0aC10by1waW4taGVscGVyPiBhZGRyZXNzXG4i CisJCSJcbiIKKwkJIlx0RGVmYXVsdCBmb3IgYWdlbnQtcGF0aDogL29yZy9ibHVlei9wYXNza2V5 X2FnZW50Xzxwcm9ncmFtIHBpZD5cbiIKIAkJIlxuIik7CiB9CiAK