Per Jessen wrote:

> My personal preference is to just not use it. That is, to effectively alter 
> the behaviour
> of OnUpdateExecute.  It would mean it could only accept actual shell-scripts 
> and
> executables, and not plain commandlines (with pipes and redirects etc.)
> execve() works with executables and scripts with #!/your/favourite/shell .
> 
> Is this too much of a change - I fully understand it'll cause some 
> incompatibility,
> but since clamav is still at a 0.8x release, prehaps it is reasonable?

http://jessen.ch/files/patch-clamav-0.83-freshclam-with-fork4

This version (now #4) does not use /bin/sh, and as such the argument provided 
for 
OnErrorExecute and OnUpdateExecute must be either an executable or a 
shell-script with 
#!/your/favourite/interpreter - followed by any number of arguments.
And of course there's nothing stopping you from using something like this: 
OnUpdateExecute = /bin/sh -c .........

Note - you'll need to rerun automake to build with this patch.


/Per Jessen, Zürich


_______________________________________________
http://lurker.clamav.net/list/clamav-devel.html

Reply via email to