Changes in golded+ sources

Michael Dukelsky написал(а) к Semen Panevin в Apr 17 22:04:12 по местному времени:

Привет, Semen!

29 Apr 17 08:04, Semen Panevin послал(а) письмо к Michael Dukelsky:

MD>>>> Скорее всего эта функция пишет в массив, не
MD>>>> проверяя нарушения границ массива, и радостно перезаписывает то
MD>>>> место, где хранится указатель на массив. После чего попытка
MD>>>> освобождения выделенной памяти приводит к краху.

SP>>> Под dest выделяется памяти src len + 1. Я пробовал увеличить в 2
SP>>> раза, не помогло.

MD>> Не надо гадать. Надо проверить, что функция XlatStr действительно
MD>> портит значение указателя на выделенную память.
SP> Увеличение буфера в ТРИ раза помогло. Значит точно портит, и точно в
SP> конце. Функция здоровая с кучей непонятной мне логики, самому
SP> разобраться в ней я ниасилю. Посему вопрос: коммитить воркароунд с
SP> увеличением буфера?

Я бы не стал. Сейчас увеличение буфера в три раза помогло, а в следующий раз надо будет увеличить в четыре раза. Я бы это гадание на кофейной гуще даже не стал называть воркэраундом.

Желаю успехов, Semen!
За сим откланиваюсь, Michael.

... node (at) f1042 (dot) ru
--- GoldED+/LNX 1.1.5-b20151128

Semen Panevin написал(а) к Vitaliy Aksyonov в Apr 17 23:17:08 по местному времени:

Доброго здоровьица тебе, Vitaliy!

Saturday April 29 2017 20:48, Vitaliy Aksyonov писал Semen Panevin:

MD>>> Не надо гадать. Надо проверить, что функция XlatStr
MD>>> действительно портит значение указателя на выделенную память.
SP>> Увеличение буфера в ТРИ раза помогло. Значит точно портит, и
SP>> точно в конце. Функция здоровая с кучей непонятной мне логики,
SP>> самому разобраться в ней я ниасилю. Посему вопрос: коммитить
SP>> воркароунд с увеличением буфера?

VA> Я могу глянуть. Постараюсь найти причину. Костыль, который вроде
VA> работает, но непонятно почему, тоже ничего хорошего.
Было бы здорово, если глянешь...

С наилучшими пожеланиями, Семён.

... В гостях хорошо, а дома хуже...
--- GoldED+/LNX 1.1.5-b20170303 (Linux 4.1.12-gentoo iF6M10)

Semen Panevin написал(а) к Michael Dukelsky в Apr 17 23:18:16 по местному времени:

Доброго здоровьица тебе, Michael!

Saturday April 29 2017 22:04, Michael Dukelsky писал Semen Panevin:

MD>>> Не надо гадать. Надо проверить, что функция XlatStr
MD>>> действительно портит значение указателя на выделенную память.
SP>> Увеличение буфера в ТРИ раза помогло. Значит точно портит, и
SP>> точно в конце. Функция здоровая с кучей непонятной мне логики,
SP>> самому разобраться в ней я ниасилю. Посему вопрос: коммитить
SP>> воркароунд с увеличением буфера?

MD> Я бы не стал. Сейчас увеличение буфера в три раза помогло, а в
MD> следующий раз надо будет увеличить в четыре раза. Я бы это гадание на
MD> кофейной гуще даже не стал называть воркэраундом.
Да вот и мне оно по тем же причинам не нравится. Поэтому и не коммичу, а спрашиваю мнения народа... Смущает, что падает пока только у меня, и началось после очередной пересборки после правок, совершенно с этим местом, вроде бы, не связанных...

С наилучшими пожеланиями, Семён.

... Стрессы начинаются с будильника... (c)...
--- GoldED+/LNX 1.1.5-b20170303 (Linux 4.1.12-gentoo iF6M10)

Vitaliy Aksyonov написал(а) к Semen Panevin в Apr 17 00:19:48 по местному времени:

Привет, Semen!

29 апр 17 23:18, Semen Panevin -> Michael Dukelsky:

MD>>>> Не надо гадать. Надо проверить, что функция XlatStr
MD>>>> действительно портит значение указателя на выделенную память.
SP>>> Увеличение буфера в ТРИ раза помогло. Значит точно портит, и
SP>>> точно в конце. Функция здоровая с кучей непонятной мне логики,
SP>>> самому разобраться в ней я ниасилю. Посему вопрос: коммитить
SP>>> воркароунд с увеличением буфера?
MD>> Я бы не стал. Сейчас увеличение буфера в три раза помогло, а в
MD>> следующий раз надо будет увеличить в четыре раза. Я бы это
MD>> гадание на кофейной гуще даже не стал называть воркэраундом.
SP> Да вот и мне оно по тем же причинам не нравится. Поэтому и не коммичу,
SP> а спрашиваю мнения народа... Смущает, что падает пока только у меня, и
SP> началось после очередной пересборки после правок, совершенно с этим
SP> местом, вроде бы, не связанных...

Покопался немного.
До конца еще не разобрался, но попробуй для начала удалить файл goldxlat.gel и попробуй воспроизвести заново.

С наилучшими пожеланиями, Vitaliy.

... 10.0 times 0.10 is hardly ever 1.00.
--- GoldED+/LNX 1.1.5-b20160201

Semen Panevin написал(а) к Vitaliy Aksyonov в Apr 17 10:37:42 по местному времени:

Доброго здоровьица тебе, Vitaliy!

Sunday April 30 2017 00:19, Vitaliy Aksyonov писал Semen Panevin:

SP>> Да вот и мне оно по тем же причинам не нравится. Поэтому и не
SP>> коммичу, а спрашиваю мнения народа... Смущает, что падает пока
SP>> только у меня, и началось после очередной пересборки после
SP>> правок, совершенно с этим местом, вроде бы, не связанных...

VA> Покопался немного.
VA> До конца еще не разобрался, но попробуй для начала удалить файл
VA> goldxlat.gel и попробуй воспроизвести заново.
Пробую... Слово два, слово три, слово пять...

Начал писать слово "работает" и всё снова упало :(

С наилучшими пожеланиями, Семён.

... В гостях хорошо, а дома хуже...
--- GoldED+/LNX 1.1.5-b20170303 (Linux 4.1.12-gentoo iF6M10)

Vitaliy Aksyonov написал(а) к Semen Panevin в Apr 17 18:49:38 по местному времени:

Привет, Semen!

30 апр 17 10:37, Semen Panevin -> Vitaliy Aksyonov:

SP>>> Да вот и мне оно по тем же причинам не нравится. Поэтому и не
SP>>> коммичу, а спрашиваю мнения народа... Смущает, что падает пока
SP>>> только у меня, и началось после очередной пересборки после
SP>>> правок, совершенно с этим местом, вроде бы, не связанных...
VA>> Покопался немного.
VA>> До конца еще не разобрался, но попробуй для начала удалить файл
VA>> goldxlat.gel и попробуй воспроизвести заново.
SP> Пробую... Слово два, слово три, слово пять...

SP> Начал писать слово "работает" и всё снова упало :(

Ок. Копну еще. Можешь сделать дама до входа в xlatstr?
В письме есть Soft CR (код 0x8D)?

С наилучшими пожеланиями, Vitaliy.

... 10.0 times 0.10 is hardly ever 1.00.
--- GoldED+/LNX 1.1.5-b20160201

Semen Panevin написал(а) к Vitaliy Aksyonov в May 17 07:53:22 по местному времени:

Доброго здоровьица тебе, Vitaliy!

Sunday April 30 2017 18:49, Vitaliy Aksyonov писал Semen Panevin:

SP>> Начал писать слово "работает" и всё снова упало :(

VA> Ок. Копну еще. Можешь сделать дама до входа в xlatstr?
VA> В письме есть Soft CR (код 0x8D)?
Да, в этом письме есть символ с кодом 0x8D. Только это не Soft CR, а русская буква Н :)

Но наличие или отсутствие этой буквы на проблему не влияет, у меня воспроизводилось и без неё.

С наилучшими пожеланиями, Семён.

... Стрессы начинаются с будильника... (c)...
--- GoldED+/LNX 1.1.5-b20170303 (Linux 4.1.12-gentoo iF6M10)

Vitaliy Aksyonov написал(а) к Semen Panevin в May 17 16:10:26 по местному времени:

Привет, Semen!

01 май 17 07:53, Semen Panevin -> Vitaliy Aksyonov:

SP>>> Начал писать слово "работает" и всё снова упало :(
VA>> Ок. Копну еще. Можешь сделать дама до входа в xlatstr?
VA>> В письме есть Soft CR (код 0x8D)?
SP> Да, в этом письме есть символ с кодом 0x8D. Только это не Soft CR, а
SP> русская буква Н :)

Просто в коде этой функции этот символ особым способом обрабатывается.

SP> Но наличие или отсутствие этой буквы на проблему не влияет, у меня
SP> воспроизводилось и без неё.

Ниже прочитал, что уже все работает?

С наилучшими пожеланиями, Vitaliy.

... 10.0 times 0.10 is hardly ever 1.00.
--- GoldED+/LNX 1.1.5-b20160201

Semen Panevin написал(а) к Vitaliy Aksyonov в May 17 19:49:54 по местному времени:

Доброго здоровьица тебе, Vitaliy!

Monday May 01 2017 16:10, Vitaliy Aksyonov писал Semen Panevin:

SP>> Да, в этом письме есть символ с кодом 0x8D. Только это не Soft
SP>> CR, а русская буква Н :)
VA> Просто в коде этой функции этот символ особым способом обрабатывается.
Если ты про case SOFTCR ... else if (CompTable) то этот else я уже пробовал полностью закомментировать, ничего не изменилось.

SP>> Но наличие или отсутствие этой буквы на проблему не влияет, у
SP>> меня воспроизводилось и без неё.

VA> Ниже прочитал, что уже все работает?
Кто прочитал, где, и что работает? Ты сабж не попутал?

С наилучшими пожеланиями, Семён.

... Учиться, учиться и учиться! (с) Ленин
--- GoldED+/LNX 1.1.5-b20170303 (Linux 4.1.12-gentoo iF6M10)

Vitaliy Aksyonov написал(а) к Semen Panevin в May 17 20:08:44 по местному времени:

Привет, Semen!

01 май 17 19:49, Semen Panevin -> Vitaliy Aksyonov:

SP>>> Да, в этом письме есть символ с кодом 0x8D. Только это не Soft
SP>>> CR, а русская буква Н :)
VA>> Просто в коде этой функции этот символ особым способом
VA>> обрабатывается.
SP> Если ты про case SOFTCR ... else if (CompTable) то этот else я уже
SP> пробовал полностью закомментировать, ничего не изменилось.

Да. Про него.
Хм. Еще тогда такое попробуй. Увеличь буфер в три раза, когда работает и посмотри, что в нем оказывается после вызова xlatstr.
Там в этой функции очень много зависит от флагов компиляции и внешних настроек.

SP>>> Но наличие или отсутствие этой буквы на проблему не влияет, у
SP>>> меня воспроизводилось и без неё.
VA>> Ниже прочитал, что уже все работает?
SP> Кто прочитал, где, и что работает? Ты сабж не попутал?

Упс. Перепутал авторов. :) Значит, продолжаем копать. :)

Все-таки можешь выслать дамп после падения и дамп перед вызовом xlatstr и бинарник нестрипнутый?

Напомни, какой спеллчекер используешь?

С наилучшими пожеланиями, Vitaliy.

... 10.0 times 0.10 is hardly ever 1.00.
--- GoldED+/LNX 1.1.5-b20160201