Summary: | irattach logs bad error message on TIOCSETD failure | ||||||
---|---|---|---|---|---|---|---|
Product: | Sisyphus | Reporter: | Sergey Vlasov <vsu> | ||||
Component: | irda-utils | Assignee: | Nobody's working on this, feel free to take it <nobody> | ||||
Status: | CLOSED FIXED | QA Contact: | qa-sisyphus | ||||
Severity: | normal | ||||||
Priority: | P2 | CC: | week | ||||
Version: | unstable | ||||||
Hardware: | all | ||||||
OS: | Linux | ||||||
Attachments: |
|
Description
Sergey Vlasov
2005-03-12 18:58:30 MSK
А можно патчик для тупых новых майнтейнеров? :) Найти ничего готового не вышло, читать daemon() чуточку выходит за рамки намеченного на ближайшие три месяца... попросил в sisyphus@, кому нетрудно. запись в stderr наверное оставлена для тех случаев. когда сервер запускается с отладкой, и stderr не закрывается. Перестановка строчек syslog и fprintf должна решить проблему. Created attachment 1615 [details] 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). Мужыки EBADF получился из-за fprintf(stderr), если fd=2 уже закрыт. Переставьте syslog и fprintf местами либо сохраняйте errno. man errno |col -b |grep -A33 ^NOTES Кстати у syslog() есть формат %m. Да уже приложил приложенное. Надо в апстрим закинуть вместе с твоей/Lost's рекомендацией. Учтено в 0.9.16-alt3 и закинуто в апстрим: http://sourceforge.net/tracker/index.php?func=detail&aid=1551560&group_id=5616&atid=305616 Всем спасибо :-) |