6257 2005-03-12 18:58:30 +0300 irattach logs bad error message on TIOCSETD failure 2006-12-17 23:18:07 +0300 1 1 4 Development Sisyphus irda-utils unstable all Linux CLOSED FIXED P2 normal --- 1 vsu nobody week qa-sisyphus oldest_to_newest 22518 0 vsu 2005-03-12 18:58:30 +0300 В irda-utils-0.9.16/irattach/irattach.c есть следующий код: if (ioctl(ttyfd, TIOCSETD, &irdadisc) < 0){ fprintf(stderr, "Maybe you don't have IrDA support in your kernel?\n"); syslog(LOG_ERR, "irattach: tty: set_disc(%d): %s\n", irdadisc, strerror(errno)); clean_exit(-1); } Если вызов TIOCSETD завершается с ошибкой (например, в случае проблем с загрузкой модуля irtty_sir), сообщение об этом должно писаться в syslog - оно туда и записывается, но с неверным кодом ошибки: irattach: tty: set_disc(11): Bad file descriptor Т.е., создаётся впечатление, что вызов TIOCSETD вернул в errno код ошибки EBADF, хотя strace показывает другую ситуацию: ioctl(3, TIOCSETD, [11]) = -1 EINVAL (Invalid argument) write(2, "Maybe you don\'t have IrDA suppor"..., 50) = -1 EBADF (Bad file descriptor) Дело в том, что ещё раньше irattach вызывает fork_now(), где в том числе закрываются все открытые ранее файловые дескрипторы, в том числе и stdin/stdout/stderr. Поэтому fprintf(stderr, ...) завершается с ошибкой, и этот код ошибки затирает код, полученный ранее от TIOCSETD. Вообще закрытие дескрипторов 0, 1, 2 чревато странными проблемами; обычно в подобных случаях открывают /dev/null (см., например, man 3 daemon). 39483 1 mike 2006-08-10 11:06:48 +0400 А можно патчик для тупых новых майнтейнеров? :) 39980 2 mike 2006-09-02 23:33:56 +0400 Найти ничего готового не вышло, читать daemon() чуточку выходит за рамки намеченного на ближайшие три месяца... попросил в sisyphus@, кому нетрудно. 39982 3 damir 2006-09-02 23:39:09 +0400 запись в stderr наверное оставлена для тех случаев. когда сервер запускается с отладкой, и stderr не закрывается. Перестановка строчек syslog и fprintf должна решить проблему. 39983 4 1615 vsu 2006-09-02 23:55:59 +0400 Created attachment 1615 irda-utils-0.9.16-alt-irattach-daemonize.patch - irattach: daemonize properly: + close fds up to sysconf(_SC_OPEN_MAX) instead of hardcoded 64; + redirect stdin/out/err to /dev/null instead of leaving them closed (#6257). 39992 5 at 2006-09-03 15:29:56 +0400 Мужыки EBADF получился из-за fprintf(stderr), если fd=2 уже закрыт. Переставьте syslog и fprintf местами либо сохраняйте errno. man errno |col -b |grep -A33 ^NOTES 39993 6 at 2006-09-03 15:46:58 +0400 Кстати у syslog() есть формат %m. 39996 7 mike 2006-09-03 21:43:35 +0400 Да уже приложил приложенное. Надо в апстрим закинуть вместе с твоей/Lost's рекомендацией. 39997 8 mike 2006-09-03 22:15:07 +0400 Учтено в 0.9.16-alt3 и закинуто в апстрим: http://sourceforge.net/tracker/index.php?func=detail&aid=1551560&group_id=5616&atid=305616 Всем спасибо :-) 1615 2006-09-02 23:55:59 +0400 2006-09-02 23:55:59 +0400 irda-utils-0.9.16-alt-irattach-daemonize.patch irda-utils-0.9.16-alt-irattach-daemonize.patch text/plain 844 vsu LS0tIGlyZGEtdXRpbHMtMC45LjE2L2lyYXR0YWNoL3V0aWwuYy5hbHQtaXJhdHRhY2gtZGFlbW9u aXplCTIwMDQtMDQtMDEgMDE6NTU6NDEgKzA0MDAKKysrIGlyZGEtdXRpbHMtMC45LjE2L2lyYXR0 YWNoL3V0aWwuYwkyMDA2LTA5LTAyIDIzOjQ3OjQwICswNDAwCkBAIC0xNTgsNiArMTU4LDcgQEAg dm9pZCBmb3JrX25vdyhpbnQgdHR5ZmQpCiB7CiAJaW50IHJldDsKIAlpbnQgaTsKKwlpbnQgZmQ7 CiAKIAlpZiAoKHJldCA9IGZvcmsoKSkgPiAwKQogCQlleGl0KDApOwpAQCAtMTczLDEwICsxNzQs MjMgQEAgdm9pZCBmb3JrX25vdyhpbnQgdHR5ZmQpCiAJfQogCiAJLyogQ2xvc2UgYWxsIG9wZW4g aW5oZXJpdGVkIGZpbGVzISBFeGNlcHQgZm9yIHR0eWZkISAqLwotCWZvciAoaSA9IDA7IGkgPCA2 NDsgaSsrKQorCWZvciAoaSA9IDA7IGkgPCBzeXNjb25mKF9TQ19PUEVOX01BWCk7IGkrKykKIAkJ aWYoaSAhPSB0dHlmZCkKIAkJCWNsb3NlKGkpOwogCisJLyogUmVkaXJlY3Qgc3RhbmRhcmQgZGVz Y3JpcHRvcnMgdG8gL2Rldi9udWxsIHRvIGF2b2lkIHRyb3VibGUgKi8KKwlmZCA9IG9wZW4oIi9k ZXYvbnVsbCIsIE9fUkRXUik7CisJaWYgKGZkID49IDApIHsKKwkJaWYgKGZkICE9IFNURElOX0ZJ TEVOTykKKwkJCWR1cDIoZmQsIFNURElOX0ZJTEVOTyk7CisJCWlmIChmZCAhPSBTVERPVVRfRklM RU5PKQorCQkJZHVwMihmZCwgU1RET1VUX0ZJTEVOTyk7CisJCWlmIChmZCAhPSBTVERFUlJfRklM RU5PKQorCQkJZHVwMihmZCwgU1RERVJSX0ZJTEVOTyk7CisJCWlmIChmZCA+IFNUREVSUl9GSUxF Tk8pCisJCQljbG9zZShmZCk7CisJfQorCiAJd3JpdGVfcGlkKCk7CiB9IAogCg==