SAMS

Ticket #57 (closed defect: fixed)

Opened 2 years ago

Last modified 4 months ago

при IP авторизации и REJIK samsdaemon выпадает в кору при реконфигурации сквида

Reported by: alex@eab.ru Assigned to: DmitryChemerik
Priority: major Milestone: sams-1.0.1
Component: demons Version:
Keywords: Cc:

Description

linux 2.6.22, gcc - 4.2.0, sams - из транка самый свежачок.. авторизация только по IP в процессе поиска обнаружилось что выпадение идет на строке 1959 samsdaemon.c

fprintf(fout,"%s/255.255.255.255\n",row2[11]&&strlen(row2[11])>4);

замена на fprintf(fout,"%s/255.255.255.255\n",row2[11]); решила проблему.

что самое интересное, оригинальный кусок кода выглядит так: fprintf(fout,"%s/255.255.255.255\n",row2[11]&&strlen(row2[11])>4); if(RREJIK==1&&atoi(row2[10])>0)

fprintf(fout2,"%s",row2[11]);

Attachments

Change History

09/19/07 22:53:25 changed by alex@eab.ru

Дополнение к предыдущему. также оказалось что список для реджика формируется в виде такого вот списка: 192.168.0.1192.168.0.2192.168.0.76

причина кроется в строке 1961: fprintf(fout2,"%s",row2[11]);

пропущен символ \n, вот так заработало как надо: fprintf(fout2,"%s\n",row2[11]);

12/02/07 04:53:55 changed by PavelVinogradov

  • milestone set to deprecated.

Воспроизводится ли ошибка на sams 1.0.0?

12/03/07 15:02:53 changed by alex@eab.ru


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

Index: trunk/samsdaemon.c
===================================================================
--- trunk/samsdaemon.c (revision 210)
+++ trunk/samsdaemon.c (working copy)
@@ -1940,7 +1940,7 @@

{

fprintf(fout,"%s/255.255.255.255\n",row2[11]);

if(RREJIK==1&&atoi(row2[10])>0)

- fprintf(fout2,"%s",row2[11]);

+ fprintf(fout2,"%s\n",row2[11]);

}

// fprintf(fout,"\n");

12/08/07 23:11:22 changed by PavelVinogradov

  • status changed from new to closed.
  • resolution set to fixed.
  • milestone changed from deprecated to sams-1.0.1.

Спасибо, исправлено в [211], [212] для trunk и 1.0.1 соответственно.

03/09/09 19:32:19 changed by Headache

<a href="http://headache-h.blogspot.com/">Headache</a>

03/11/09 05:23:30 changed by Skin

<a href="http://skin-care-tips4all.blogspot.com/">Synalar </a>

Add/Change #57 (при IP авторизации и REJIK samsdaemon выпадает в кору при реконфигурации сквида)




Action