[devel] #9218 status

[devel] #9218 status

[Anton Farygin]

К сожалению, у меня сейчас нет возможности исправить dansguardian для 
нового clamav.

Поэтому, есть такое предложение (Андрей, выполни пожалуйста эту команду):

$ ssh git.alt acl sisyphus dansguardian add @everybody
dansguardian: Permission denied, only liks is allowed to change this acl

Re: [devel] #9218 status (clamav)

[Sergey Y. Afonin]

On Thursday 16 July 2009, Anton Farygin wrote:

> К сожалению, у меня сейчас нет возможности исправить dansguardian для 
> нового clamav.

На самом деле я сейчас вот сижу и думаю, может, начать паковать clamav
с версией в имени пакета ?

-- 
С уважением, Сергей Афонин
asy@altlinux.ru

Re: [devel] #9218 status (clamav)

[Max Ivanov]

>
> На самом деле я сейчас вот сижу и думаю, может, начать паковать clamav
> с версией в имени пакета ?

Давайте не будем разводить зоопарк на каждый чих? Если с BerkleyDB это
оправдано, но мильйон clamav мне лично видеть бы не хотелось

Re: [devel] #9218 status (clamav)

[Sergey Y. Afonin]

On Thursday 16 July 2009, Max Ivanov wrote:

> > На самом деле я сейчас вот сижу и думаю, может, начать паковать clamav
> > с версией в имени пакета ?
> 
> Давайте не будем разводить зоопарк на каждый чих? Если с BerkleyDB это
> оправдано, но мильйон clamav мне лично видеть бы не хотелось
 
С одной стороны да, а, с другой, исправление критических обновлений выливается
в многомесячную борьбу. При этом, критические обновления в ClamAV достаточно
регулярны. :-(

-- 
С уважением, Сергей Афонин
asy@altlinux.ru

Re: [devel] #9218 status (clamav)

[Sergey Y. Afonin]

On Thursday 16 July 2009, Sergey Y. Afonin wrote:

> При этом, критические обновления в ClamAV достаточно регулярны. :-(

И ещё. Лично мне тут интересен только clamd: всё, чем пользуюсь я,
использует работу через сокет, а не через libclamav. И, в отличие
от libclamav, формат работы через сокет меняется значительно реже,
собственно, я и не помню такого, чтобы что-то работать перестало.

-- 
С уважением, Сергей Афонин
asy@altlinux.ru

Re: [devel] #9218 status (clamav)

[Max Ivanov]

>
> С одной стороны да, а, с другой, исправление критических обновлений выливается
> в многомесячную борьбу. При этом, критические обновления в ClamAV достаточно
> регулярны. :-(

Это говорит о том, что для  пересборки с Release+1 любых зависимых
пакетов не надо никакой процедуры NMU, это должен уметь делать любой,
кто заливает библиотеку от которой зависят другие пакеты.

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

Re: [devel] #9218 status (clamav)

[Sergey Y. Afonin]

On Thursday 16 July 2009, Max Ivanov wrote:

> > С одной стороны да, а, с другой, исправление критических обновлений выливается
> > в многомесячную борьбу. При этом, критические обновления в ClamAV достаточно
> > регулярны. :-(
> 
> Это говорит о том, что для  пересборки с Release+1 любых зависимых
> пакетов не надо никакой процедуры NMU, это должен уметь делать любой,
> кто заливает библиотеку от которой зависят другие пакеты.
 
Несовсем. Вот, например, python-module-pyclamav надо или чинить, или
выкидывать совсем. Это уже не Release+1. Новой версии dansguardian
нет тоже...

-- 
С уважением, Сергей Афонин
asy@altlinux.ru

Re: [devel] #9218 status (clamav, dansguardian)

[Anton Farygin]

Sergey Y. Afonin пишет:
> On Thursday 16 July 2009, Max Ivanov wrote:
> 
>>> С одной стороны да, а, с другой, исправление критических обновлений выливается
>>> в многомесячную борьбу. При этом, критические обновления в ClamAV достаточно
>>> регулярны. :-(
>> Это говорит о том, что для  пересборки с Release+1 любых зависимых
>> пакетов не надо никакой процедуры NMU, это должен уметь делать любой,
>> кто заливает библиотеку от которой зависят другие пакеты.
>  
> Несовсем. Вот, например, python-module-pyclamav надо или чинить, или
> выкидывать совсем. Это уже не Release+1. Новой версии dansguardian
> нет тоже...

Меня уже кто-то просил выкинуть поддержку clamav из dansguardian.

И, если я не ошибаюсь, upstream планирует сделать тоже-самое.

Кому-то нужен clamav в dansguardian ?

Re: [devel] #9218 status (clamav, dansguardian)

[Sergey Y. Afonin]

On Thursday 16 July 2009, Anton Farygin wrote:

> И, если я не ошибаюсь, upstream планирует сделать тоже-самое.
> 
> Кому-то нужен clamav в dansguardian ?
 
По мне, так вполне хватит clamd. Тем более, что тут наблюдается
стабильность протокола.

-- 
С уважением, Сергей Афонин
asy@altlinux.ru

Re: [devel] #9218 status (clamav, dansguardian)

[Andrew Avramenko]

> Меня уже кто-то просил выкинуть поддержку clamav из dansguardian.
>
> И, если я не ошибаюсь, upstream планирует сделать тоже-самое.
>
> Кому-то нужен clamav в dansguardian ?


Если clamd при этом останется, то вполне нормально. Если нет, то
ценность dansguardian упадет в два раза. У меня сейчас нет никакого
продакшна с линуксом, поэтому я в ближайшее время собирать не буду. Не
думаю, что кому-то нужны синтетические пакеты, работоспособность
которых проверяется простыми установкой и запуском.


-- 
With best regards,
Andrew

Re: [devel] #9218 status (clamav)

[Sergey Y. Afonin]

On Thursday 16 July 2009, Sergey Y. Afonin wrote:

> Вот, например, python-module-pyclamav надо или чинить, или
> выкидывать совсем.

С p3scan тоже вопрос. Новой версии нет, с clamd работать не умеет.
pyclamav хоть на сайте выкинуть советуют, а dansguardian с clamd
работать умеет... 

-- 
С уважением, Сергей Афонин
asy@altlinux.ru

Re: [devel] #9218 status (clamav)

[Sergey Y. Afonin]

On Thursday 16 July 2009, Sergey Y. Afonin wrote:

> > Вот, например, python-module-pyclamav надо или чинить, или
> > выкидывать совсем.
 
> С p3scan тоже вопрос. Новой версии нет, с clamd работать не умеет.

Похоже надо p3scan тоже, пока, убирать. Действительно, держать 2 версии
clamav не хорошо: вроде как получается, что нет никакой гарантии, что
старая библиотека будет работать с базой, которую будет вытягивать
новый freshclam. :-( Или надо паковать из расчёта, что должны одновременно
присутствовать две базы и два freshclam.

-- 
С уважением, Сергей Афонин
asy@altlinux.ru

Re: [devel] #9218 status (clamav)

[Sergey Y. Afonin]

On Saturday 18 July 2009, Sergey Y. Afonin wrote:

> Похоже надо p3scan тоже, пока, убирать.

И в рассылке последнее сообщение от апреля прошлого года, да и за весь
2008 только два, оба анонсы новых версий.

-- 
С уважением, Сергей Афонин
asy@altlinux.ru

Re: [devel] #9218 status (clamav)

[Vladimir Lettiev]

[-- Attachment #1: Type: text/plain, Size: 749 bytes --]

On Thu, Jul 16, 2009 at 05:17:59PM +0500, Sergey Y. Afonin wrote:
> On Thursday 16 July 2009, Sergey Y. Afonin wrote:
> 
> > Вот, например, python-module-pyclamav надо или чинить, или
> > выкидывать совсем.
> 
> С p3scan тоже вопрос. Новой версии нет, с clamd работать не умеет.

Говорят, что можно использовать clamdscan:
http://wiki.clamav.net/bin/view/Main/FAQ#How_do_I_use_ClamAV_with_p3scan

Вероятно нужно выключить опцию сборки с библиотекой libclamav и воспользоваться
альтернативным интерфейсом.

-- 
Vladimir Lettiev aka crux

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 197 bytes --]

Re: [devel] #9218 status (clamav)

[Sergey Y. Afonin]

On Monday 20 July 2009, Vladimir Lettiev wrote:

> Говорят, что можно использовать clamdscan:
> http://wiki.clamav.net/bin/view/Main/FAQ#How_do_I_use_ClamAV_with_p3scan
 
Вообще да, если написать что-то вроде 

=virusregexp = .*: (.*)
  scanner = /usr/bin/clamdscan --no-summary -i 
  scannertype = basic=

возможно сработает. На сколько я понял, при использовании libclamav он
безусловно проверяет ? После кого-то ведь смысла нет особенного...
Кто бы попробовал ?

-- 
С уважением, Сергей Афонин

Re: [devel] #9218 status (clamav)

[Michael Shigorin]

On Thu, Jul 16, 2009 at 12:02:48PM +0500, Sergey Y. Afonin wrote:
> На самом деле я сейчас вот сижу и думаю, может, начать паковать
> clamav с версией в имени пакета ?

На старый можно будет хоть сразу автоматом блокеры по части
security вешать... :(

-- 
 ---- WBR, Michael Shigorin <mike@altlinux.ru>
  ------ Linux.Kiev http://www.linux.kiev.ua/

Re: [devel] #9218 status (clamav)

[Sergey Y. Afonin]

On Monday 20 July 2009, Michael Shigorin wrote:

> > На самом деле я сейчас вот сижу и думаю, может, начать паковать
> > clamav с версией в имени пакета ?
> 
> На старый можно будет хоть сразу автоматом блокеры по части
> security вешать... :(

Тоже верно... В общем, как dansguardian пересоберётся, останется только
вопрос с p3scan. Тут проблем две. Одна, как я написал, невозможность
его работы с новым ClamAV, а вторая - Алексей Шабалин мне в четверг
написал, что в отпуск уходит. Вопрос выбрасывания пакта из Сизифа без
него решать, наверное, не сильно хорошо... Хотя, с другой стороны, если
насильно ClamAV пересобрать, работать не будет всё-равно...

-- 
С уважением, Сергей Афонин
asy@altlinux.ru

Re: [devel] #9218 status (clamav)

[Michael Shigorin]

[-- Attachment #1: Type: text/plain, Size: 652 bytes --]

On Mon, Jul 20, 2009 at 10:07:03AM +0500, Sergey Y. Afonin wrote:
> Тоже верно... В общем, как dansguardian пересоберётся,
> останется только вопрос с p3scan. Тут проблем две. Одна, как я
> написал, невозможность его работы с новым ClamAV, а вторая -
> Алексей Шабалин мне в четверг написал, что в отпуск уходит.
> Вопрос выбрасывания пакта из Сизифа без него решать, наверное,
> не сильно хорошо... Хотя, с другой стороны, если насильно
> ClamAV пересобрать, работать не будет всё-равно...

Думаю, при прочих вышеозвученных -- выкидывать.

-- 
 ---- WBR, Michael Shigorin <mike@altlinux.ru>
  ------ Linux.Kiev http://www.linux.kiev.ua/

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: [devel] #9218 status

[Andrew Avramenko]

done.

16 июля 2009 г. 10:56 пользователь Anton Farygin (rider@altlinux.com) написал:
> К сожалению, у меня сейчас нет возможности исправить dansguardian для нового
> clamav.
>
> Поэтому, есть такое предложение (Андрей, выполни пожалуйста эту команду):
>
> $ ssh git.alt acl sisyphus dansguardian add @everybody
> dansguardian: Permission denied, only liks is allowed to change this acl
>
>



-- 
With best regards,
Andrew

Re: [devel] #9218 status

[Sergey Y. Afonin]

On Thursday 16 July 2009, Andrew Avramenko wrote:

> > К сожалению, у меня сейчас нет возможности исправить dansguardian для нового
> > clamav.
> >
> > Поэтому, есть такое предложение (Андрей, выполни пожалуйста эту команду):
> >
> > $ ssh git.alt acl sisyphus dansguardian add @everybody
> > dansguardian: Permission denied, only liks is allowed to change this acl

> done.

Залил вчера в incoming dansguardian-2.10.1.1-alt3.src.rpm без --enable-clamav,
но что-то не вижу результата сборки. Заливал днём.

-- 
С уважением, Сергей Афонин
asy@altlinux.ru

Re: [devel] #9218 status

[REAL]

Sergey Y. Afonin пишет:
> Залил вчера в incoming dansguardian-2.10.1.1-alt3.src.rpm без --enable-clamav,
> но что-то не вижу результата сборки. Заливал днём.

Вчера girar-builder был занят другими делами (как всегда, как назло, 
тотальная пересборка именно в воскресенье).

 > ssh git.alt task ls --all|grep dansguardian 

#9557 AWAITING sisyphus/asy srpm=dansguardian-2.10.1.1-alt3.src.rpm

-- 

REAL aka Евгений Ростовцев, программист ЦНИТ КемГУ

Re: [devel] #9218 status

[Sergey Y. Afonin]

On Monday 20 July 2009, REAL wrote:

> #9557 AWAITING sisyphus/asy srpm=dansguardian-2.10.1.1-alt3.src.rpm
 
Блин, его Rider уже успел через Git пособирать... Что-то я стормозил,
srpm уже не годится...

-- 
С уважением, Сергей Афонин
asy@altlinux.ru

Re: [devel] #9218 status

[Sergey Y. Afonin]

On Monday 20 July 2009, Victor Forsyuk wrote:

> Rider, кажется, в отъезде или я ошибаюсь?
> Фтопку dansguradian или будем ждать еще месяц? :-)
 
Нет, у dansguardian @nobody в acl, учусь NMU делать по-новому. :-)

-- 
С уважением, Сергей Афонин
asy@altlinux.ru

Re: [devel] #9218 status

[Sergey Y. Afonin]

On Thursday 16 July 2009, Anton Farygin wrote:

> К сожалению, у меня сейчас нет возможности исправить dansguardian для 
> нового clamav.
 
  --enable-clamav[=no]    Enable support for the libClamAV content scanner.
                          Please note that the ClamD-compatible scanner is
                          always built, and is preferable to this plugin for
                          most usage scenarios.
  --enable-clamd[=no]     Enable support for the ClamD content scanner

Предлагаю собрать без --enable-clamav.

-- 
С уважением, Сергей Афонин
asy@altlinux.ru

Re: [devel] #9218 status

[Sergey Y. Afonin]

Задание, фактически, готово к сборке. Осталось сделать

ssh git.alt task add #9218 del p3scan

У меня на это прав нет. Если ждём возвращения Алексея Шаблина, надо
бы время жизни задания увеличить недель на несколько, я не знаю,
до какого числа он в отпуске.

-- 
С уважением, Сергей Афонин

Re: [devel] #9218 status

[Sergey Y. Afonin]

On Monday 20 July 2009, Sergey Y. Afonin wrote:

> Задание, фактически, готово к сборке. Осталось сделать
> 
> ssh git.alt task add #9218 del p3scan
> 
> У меня на это прав нет.

Добавил NMU с --disable-clamav: srpm=p3scan-3.0-alt0.2.rc1.src.rpm
Надо бы зааппрувить хотябы так пока, если никто не возьмётся проверить
http://wiki.clamav.net/bin/view/Main/FAQ#How_do_I_use_ClamAV_with_p3scan
и сделать соответствующие правки в конфиге.

-- 
С уважением, Сергей Афонин

Re: [devel] #9218 status

[Dmitry V. Levin]

[-- Attachment #1: Type: text/plain, Size: 737 bytes --]

On Tue, Jul 21, 2009 at 12:04:25AM +0500, Sergey Y. Afonin wrote:
> On Monday 20 July 2009, Sergey Y. Afonin wrote:
> 
> > Задание, фактически, готово к сборке. Осталось сделать
> > 
> > ssh git.alt task add #9218 del p3scan
> > 
> > У меня на это прав нет.
> 
> Добавил NMU с --disable-clamav: srpm=p3scan-3.0-alt0.2.rc1.src.rpm
> Надо бы зааппрувить хотябы так пока, если никто не возьмётся проверить
> http://wiki.clamav.net/bin/view/Main/FAQ#How_do_I_use_ClamAV_with_p3scan
> и сделать соответствующие правки в конфиге.

Надо сперва собрать.  Зачем апрувить то, что ещё не собрано?
В виде исключения я выполнил task approve 9218 9, не глядя на содержимое,
поскольку глядеть на него никто не собирался.


-- 
ldv

[-- Attachment #2: Type: application/pgp-signature, Size: 197 bytes --]

Re: [devel] #9218 status

[Sergey Y. Afonin]

On Tuesday 21 July 2009, Dmitry V. Levin wrote:

> > Добавил NMU с --disable-clamav: srpm=p3scan-3.0-alt0.2.rc1.src.rpm
> > Надо бы зааппрувить хотябы так пока, если никто не возьмётся проверить
> > http://wiki.clamav.net/bin/view/Main/FAQ#How_do_I_use_ClamAV_with_p3scan
> > и сделать соответствующие правки в конфиге.
> 
> Надо сперва собрать.  Зачем апрувить то, что ещё не собрано?

Понял на будущее.

-- 
С уважением, Сергей Афонин
asy@altlinux.ru

Re: [devel] #9218 status

[Vladimir Lettiev]

[-- Attachment #1.1: Type: text/plain, Size: 1465 bytes --]

On Tue, Jul 21, 2009 at 12:04:25AM +0500, Sergey Y. Afonin wrote:
> On Monday 20 July 2009, Sergey Y. Afonin wrote:
> 
> > Задание, фактически, готово к сборке. Осталось сделать
> > 
> > ssh git.alt task add #9218 del p3scan
> > 
> > У меня на это прав нет.
> 
> Добавил NMU с --disable-clamav: srpm=p3scan-3.0-alt0.2.rc1.src.rpm
> Надо бы зааппрувить хотябы так пока, если никто не возьмётся проверить
> http://wiki.clamav.net/bin/view/Main/FAQ#How_do_I_use_ClamAV_with_p3scan
> и сделать соответствующие правки в конфиге.

В соответствии с http://wiki.clamav.net/bin/view/Main/UpgradeNotes095 подправил
код p3scan для работы с новым API libclamav.
После чего у меня он собрался с новым libclamav, запустился и сообщил в логах
об успешной загрузке баз (кстати, важно поставить для p3scan зависимость на
clamav-freshlam). Большего протестировать не могу.

p.s. В clamav можно убрать зависимость на LibGMP, т.к. с версии 0.95 используется
библиотека LibTomMath (копия идёт в составе clamav)

-- 
Vladimir Lettiev aka crux

[-- Attachment #1.2: p3scan-libclamav095_api.patch --]
[-- Type: text/x-diff, Size: 5407 bytes --]

diff --git a/p3scan-3.0_rc1/src/p3scan.c b/p3scan-3.0_rc1/src/p3scan.c
index 6b85b79..bd7e149 100644
--- a/p3scan-3.0_rc1/src/p3scan.c
+++ b/p3scan-3.0_rc1/src/p3scan.c
@@ -182,7 +182,7 @@ static struct cl_stat *dbstat = NULL;
 //struct cl_stat dbstat;
 const struct cfgstruct *copt;
 struct cl_engine *engine = NULL;
-struct cl_limits limits;
+/* struct cl_limits limits; */
 struct cl_cvd *daily;
 const char *dbdir;
 unsigned int sigs = 0;
@@ -281,7 +281,7 @@ void do_sigterm_main(int signr){
   }
   if((ret=unlink(config->pidfile)!=0)) do_log(LOG_NOTICE, "ERR: Unable to remove %s", config->pidfile);
 #if HAVE_CLAMAV
-  if (engine) cl_free(engine);
+  if (engine) cl_engine_free(engine);
 #endif
   do_log(LOG_NOTICE, PROGRAM " terminates now");
   exit(EXIT_SUCCESS);
@@ -536,9 +536,15 @@ int init_clamav(void){
   mvold=0;
   if (config->clamav){
     if(engine){
-      cl_free(engine);
+      cl_engine_free(engine);
       engine = NULL;
     }
+    if (cl_init(CL_SCAN_STDOPT) != CL_SUCCESS) {
+      do_log(LOG_EMERG, "ERR: cant init libclamav");
+      return SCANNER_RET_ERR;
+    } else {
+      engine = cl_engine_new();
+    }
     dbdir=cl_retdbdir();
 /*
     // Check for updated info first, if we find it, then check for old data and mv it so it is
@@ -566,27 +572,27 @@ int init_clamav(void){
     if(dbstat == NULL){
       dbstat = (struct cl_stat *) malloc(sizeof(struct cl_stat));
       if(!dbstat) {
-        cl_free(engine);
+        cl_engine_free(engine);
         do_log(LOG_EMERG, "ClamAV can't allocate memory for dbstat ERR: %s",cl_strerror(ret));
         return SCANNER_RET_ERR;
       }
     } else {
       if((ret = cl_statfree(dbstat))){
-        if (engine) cl_free(engine);
+        if (engine) cl_engine_free(engine);
         do_log(LOG_EMERG, "ClamAV Database statfree ERR: %s",cl_strerror(ret));
         return SCANNER_RET_ERR;
       }
     }
     //if((ret = cl_loaddbdir(dbdir, &engine, &sigs))) {
-    if((ret = cl_load(dbdir, &engine, &sigs, CL_DB_STDOPT))) {
-      if (engine) cl_free(engine);
+    if((ret = cl_load(dbdir, engine, &sigs, CL_DB_STDOPT))) {
+      if (engine) cl_engine_free(engine);
       if (dbstat) cl_statfree(dbstat);
       do_log(LOG_EMERG, "ERR: %s",cl_strerror(ret));
       return SCANNER_RET_ERR;
     }
     // build engine
-    if((ret = cl_build(engine))){
-      if (engine) cl_free(engine);
+    if((ret = cl_engine_compile(engine))){
+      if (engine) cl_engine_free(engine);
       if (dbstat) cl_statfree(dbstat);
 // Emergency: init_clamav. could not init.
       do_log(LOG_EMERG, "ClamAV Database build ERR: %s",cl_strerror(ret));
@@ -595,15 +601,19 @@ int init_clamav(void){
     memset(dbstat, 0, sizeof(struct cl_stat));
     cl_statinidir(dbdir, dbstat);
     // set up archive limits (ClamAV Defaults)
-    memset(&limits, 0, sizeof(struct cl_limits));
-    limits.maxfilesize = 25 * 1048576; // compressed files will only be decompressed and scanned up to this size
-    limits.maxreclevel = 16; // maximum recursion level for archives 
-    limits.maxfiles = 1000; // maximum number of files to be scanned within a single archive
+   /* memset(&limits, 0, sizeof(struct cl_limits)); */
+    cl_engine_set_num(engine,CL_ENGINE_MAX_FILESIZE, 25 * 1048576);
+    /* limits.maxfilesize = 25 * 1048576; // compressed files will only be decompressed and scanned up to this size */
+    cl_engine_set_num(engine,CL_ENGINE_MAX_RECURSION, 16);
+    /* limits.maxreclevel = 16; // maximum recursion level for archives */
+    cl_engine_set_num(engine,CL_ENGINE_MAX_FILES, 1000);
+    /* limits.maxfiles = 1000; // maximum number of files to be scanned within a single archive */
 #ifdef HAVE_MAXSCANSIZE
-    limits.maxscansize = 100 * 1048576; // during the scanning of archives this size will never be exceeded
+    cl_engine_set_num(engine,CL_ENGINE_MAX_SCANSIZE, 100 * 1048576);
+    /* limits.maxscansize = 100 * 1048576; // during the scanning of archives this size will never be exceeded */
 #else
-    limits.maxmailrec = 64; // maximum recursion level for mail files
-    limits.maxratio = 200; // maximum compression ratio
+    /* limits.maxmailrec = 64; // maximum recursion level for mail files */
+    /* limits.maxratio = 200; // maximum compression ratio */
 #endif
   }
   if (reload){
@@ -761,7 +771,7 @@ int scan_directory(struct proxycontext *p){
         return SCANNER_RET_CRIT;
       }
       /* scan descriptor */
-      if ((viret = cl_scandesc(fdc, &virnamec, &blocks, engine, &limits, CL_SCAN_STDOPT)) == CL_VIRUS){
+      if ((viret = cl_scandesc(fdc, &virnamec, &blocks, engine, CL_SCAN_STDOPT)) == CL_VIRUS){
         p->virinfo=strdup(virnamec);
         viret=SCANNER_RET_VIRUS;
       } else if (viret != CL_CLEAN) do_log(LOG_CRIT,"ERR: %s", cl_strerror(ret));
@@ -1170,7 +1180,7 @@ int scan_mailfile(struct proxycontext *p){
         return SCANNER_RET_CRIT;
       }
       /* scan descriptor */
-      if ((viret = cl_scandesc(fdc, &clamvirname, &blocks, engine, &limits, CL_SCAN_STDOPT)) == CL_VIRUS){
+      if ((viret = cl_scandesc(fdc, &clamvirname, &blocks, engine, CL_SCAN_STDOPT)) == CL_VIRUS){
         p->virinfo=strdup(clamvirname);
         viret=SCANNER_RET_VIRUS;
       } else if (viret != CL_CLEAN) do_log(LOG_CRIT,"ERR: %s", cl_strerror(ret));

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 197 bytes --]

Re: [devel] #9218 status

[Sergey Y. Afonin]

On Tuesday 21 July 2009, Vladimir Lettiev wrote:

> В соответствии с http://wiki.clamav.net/bin/view/Main/UpgradeNotes095
> подправил код p3scan для работы с новым API libclamav.
> После чего у меня он собрался с новым libclamav, запустился и сообщил
> в логах об успешной загрузке баз (кстати, важно поставить для p3scan
> зависимость на clamav-freshlam). Большего протестировать не могу.

Я, пока, запустил на сборку, как есть, без libclamav. Может быть кто-нибудь
проверит в работе, кто пользуется.

> p.s. В clamav можно убрать зависимость на LibGMP, т.к. с версии 0.95
> используется библиотека LibTomMath (копия идёт в составе clamav)

В следующий раз (и, наверное, Виктору уже), а то задание переделывать
придётся неизвестно сколько снова...

-- 
С уважением, Сергей Афонин
asy@altlinux.ru