Bug 5097

Summary: support for x86_64
Product: Sisyphus Reporter: Kachalov Anton <mouse>
Component: pam0_passwdqcAssignee: 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 Flags
multilib support
none
wrong prev attach type none

Description Kachalov Anton 2004-08-30 20:32:50 MSD
support for x86_64
Comment 1 Kachalov Anton 2004-08-30 20:33:17 MSD
Created attachment 581 [details]
multilib support

updated warnings patch
Comment 2 Kachalov Anton 2004-08-30 20:38:51 MSD
Created attachment 584 [details]
wrong prev attach type
Comment 3 Dmitry V. Levin 2004-09-01 20:39:45 MSD
Не понял смысл патчить passwdqc_random.c
Comment 4 Kachalov Anton 2004-09-02 00:21:23 MSD
У меня выдавался 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) {
Comment 5 Dmitry V. Levin 2005-02-03 22:48:14 MSK
А как именно ругается?
Comment 6 Kachalov Anton 2005-02-03 22:57:33 MSK
passwdqc_random.c: In function `_passwdqc_random':
passwdqc_random.c:69: warning: comparison between signed and unsigned
Comment 7 Kachalov Anton 2005-02-03 23:12:06 MSK
могу сказать большее, я где-то в исходниках gcc (если не ошибаюсь), видел, что
для моей архитектуры (и не тока, вроде) sizeof может вернуть отрицательное число
Comment 8 Dmitry V. Levin 2005-02-04 00:16:53 MSK
sizeof возвращает значение типа size_t (это такое достаточно большое целое без
знака).
Понятно, (end - start) имеет тип ptrdiff_t, который у тебя толще чем unsigned
int.  Попробуй сменить тип переменной length на size_t.
Comment 9 Kachalov Anton 2005-02-04 01:43:31 MSK
да, эт помогло.
- unsigned int length;
+ size_t length;

сделаешь? :)
Comment 10 Dmitry V. Levin 2005-02-09 18:36:11 MSK
Applied in -alt2.
Comment 11 Kachalov Anton 2005-02-11 19:25:56 MSK
thanks