Ticket #57 (closed defect: fixed)

Opened 2 years ago

Last modified 6 months ago

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

Reported by: alex@… Owned by: 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

Changed 2 years ago by alex@…

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

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

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

Changed 2 years ago by PavelVinogradov

  • milestone set to deprecated

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

Changed 2 years ago by alex@…


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

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");

Changed 2 years ago 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 соответственно.

Changed 12 months ago by Headache

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

Changed 12 months ago by Skin

Changed 6 months ago by relaxants

<a href=" http://muscle-relaxers-drugs.com/">muscle relaxants</a>

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

Author


E-mail address and user name can be saved in the Preferences.


Action
as closed
Next status will be 'reopened'
 
Note: See TracTickets for help on using tickets.