Bug 20842

Summary: не собирается
Product: Sisyphus Reporter: real@altlinux.org <real.altlinux.org>
Component: python-module-pygtkAssignee: Vitaly Lipatov <lav>
Status: CLOSED FIXED QA Contact: qa-sisyphus
Severity: blocker    
Priority: P3 CC: antohami, cas, cow, evg, george, grenka, imz, kotopesutility, lav, nbr, qa_viy, rider, sem, shaba, sin, slev, toni, vitty, viy
Version: unstable   
Hardware: all   
OS: Linux   
Attachments:
Description Flags
no buffer overflow
none
Вторая попытка none

Description real@altlinux.org 2009-07-21 16:43:29 MSD
Сборка вылетает с сообщением "always overflow destination buffer".

Из-за http://www.altlinux.org/Python_2.6/Rebuild ставлю blocker.
Comment 1 real@altlinux.org 2009-07-23 15:33:05 MSD
Дополнение:

На i586 собралось, достаточным оказалось добавить в спек:
%set_automake_version 1.10

Так что проблема в сборке под x86_64
Comment 2 real@altlinux.org 2009-07-24 13:48:18 MSD
Created attachment 3690 [details]
no buffer overflow

Пробую починить, просьба прокомментировать.
Comment 3 Alexey Rusakov 2009-07-24 13:56:09 MSD
Не надо так делать %-) Если не ровен день изменится размер GdkEventClient::data.b, этот код снова посыплется.
Попробуйте для начала заменить (char *) &event->client.data на event->client.data.b. И в аргументе sizeof тоже добавьте .b . Вдруг поможет.
Comment 4 real@altlinux.org 2009-07-24 14:09:55 MSD
Ошибочка вышла. Вот так собирается, вроде:

if (strlen((char*)tmp) > 20)
  ((char*)&event->client.data)[20] = '\0';

"Попробуйте для начала заменить (char *) &event->client.data на
event->client.data.b. И в аргументе sizeof тоже добавьте .b . Вдруг поможет."

Сейчас попробую.
Comment 5 real@altlinux.org 2009-07-24 14:41:43 MSD
Created attachment 3691 [details]
Вторая попытка

Вот что получилось.

Ну или (мне так проще):
http://git.altlinux.org/people/real/packages/python-module-pygtk.git
Comment 6 real@altlinux.org 2009-07-24 15:07:54 MSD
Вот здесь:

http://lists.altlinux.org/pipermail/devel/2009-July/173499.html

ещё одно возможное решение. Если это будет лучше, я добавлю его в новый коммит на git.alt.
Comment 7 Alexey Rusakov 2009-07-24 18:01:48 MSD
(В ответ на комментарий №5)
> Created an attachment (id=3691) [details]
> Вторая попытка
> 
> Вот что получилось.
Специально забивать \0 не нужно, строго говоря, это произвол. Вариант vsu@ громоздкий и параноидальный, но, конечно, самый корректный, какой только может быть :)
Comment 8 real@altlinux.org 2009-07-24 20:26:44 MSD
Хорошо, положил на git.alt

2.14.1-alt1.3

Если это всё, просьба принять этот вариант.
Comment 9 real@altlinux.org 2009-12-01 08:36:40 MSK
Уже неактуально, 2.16.0-alt4 собирается.