[Pkg-clamav-devel] Bug#802311: clamav-daemon: Please add another restorecon to the init script for SE Linux support

[Russell Coker]

On Fri, 23 Oct 2015 06:47:05 AM Andreas Cadhalpun wrote:
> > --- /etc/init.d/clamav-daemon.orig	2015-10-19 22:07:04.174298790 +1100
> > +++ /etc/init.d/clamav-daemon	2015-10-19 22:09:45.079895382 +1100
> > @@ -94,6 +94,7 @@
> > 
> >      ucf --three-way --debconf-ok "$sourcefile" "$configfile"
> >    
> >    else
> >    
> >      [ -d /var/lib/ucf/cache ] || mkdir -p /var/lib/ucf/cache
> > 
> > +    pathfind restorecon && restorecon /var/lib/ucf/cache
> > 
> >      cp $sourcefile $configfile
> >      add_to_ucf $configfile $ucffile
> >    
> >    fi
> 
> However, this patches the installed init script, not the source ...

True.  But I'm used to package maintainers not accepting patches exactly as I 
send them.  So it's more of a suggested solution than something I expected you 
to just apply.

> ... and this change is in the ucf_upgrade_check function, which
> is not used by the init script at all ...
> 
> > Init: systemd (via /run/systemd/system)
> 
> ... but your using systemd anyway.
> 
> So I'm wondering why you created this patch.

Because I encountered mis labelling and that appeared to be the cause of it.

> Nonetheless, the ucf_upgrade_check is used in the maintainer scripts,
> so I applied the change anyway, as it seems the correct thing to do.

The ucf_upgrade_check in the postinst might be the real cause and in any case 
it needs to be fixed too.  Sorry for the bad bug report.

-- 
My Main Blog         http://etbe.coker.com.au/
My Documents Blog    http://doc.coker.com.au/