Summary: | support for x86_64 | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | Sisyphus | Reporter: | Kachalov Anton <mouse> | ||||||
Component: | pam0_passwdqc | Assignee: | Dmitry V. Levin <ldv> | ||||||
Status: | CLOSED FIXED | QA Contact: | qa-sisyphus | ||||||
Severity: | normal | ||||||||
Priority: | P2 | CC: | ldv, placeholder | ||||||
Version: | unstable | ||||||||
Hardware: | all | ||||||||
OS: | Linux | ||||||||
Attachments: |
|
Description
Kachalov Anton
2004-08-30 20:32:50 MSD
Created attachment 581 [details]
multilib support
updated warnings patch
Created attachment 584 [details]
wrong prev attach type
Не понял смысл патчить passwdqc_random.c У меня выдавался warning + Werror на строчку: if (length + (end - start) >= sizeof(output) - 1) { только что-то не тот патч попал: -if (length + (end - start) >= sizeof(output) - 1) { +if (length + (end - start) >= (unsigned int) sizeof(output) - 1) { А как именно ругается? passwdqc_random.c: In function `_passwdqc_random': passwdqc_random.c:69: warning: comparison between signed and unsigned могу сказать большее, я где-то в исходниках gcc (если не ошибаюсь), видел, что для моей архитектуры (и не тока, вроде) sizeof может вернуть отрицательное число sizeof возвращает значение типа size_t (это такое достаточно большое целое без знака). Понятно, (end - start) имеет тип ptrdiff_t, который у тебя толще чем unsigned int. Попробуй сменить тип переменной length на size_t. да, эт помогло. - unsigned int length; + size_t length; сделаешь? :) Applied in -alt2. thanks |