Bug 152365 - mail/assp: Periodic script resets file ownership to root on assp db
Summary: mail/assp: Periodic script resets file ownership to root on assp db
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: Normal Affects Only Me
Assignee: Olli Hauer
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-11-18 15:00 UTC by Marco Walraven
Modified: 2011-01-07 10:40 UTC (History)
0 users

See Also:


Attachments
assp.diff (3.13 KB, patch)
2010-12-21 18:19 UTC, rnejdl
no flags Details | Diff
assp.diff (852 bytes, patch)
2010-12-21 18:25 UTC, rnejdl
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Marco Walraven 2010-11-18 15:00:17 UTC
/usr/local/etc/periodic/daily/510.assp does ASSP's nightly housekeeping.
The rebuildspamdb.pl script it calls generates new db files however if you
are running your assp as user assp these will be set to root and generate
errors during normal operation of ASSP.

Nov-18-10 13:07:00 Cleaning up penalty records ...;
Nov-18-10 13:07:00 Error: orderedtie is unable to write to /var/db/assp/pb/pbdb.black.db - Permission denied;
Nov-18-10 13:07:00 Error: orderedtie is unable to write to /var/db/assp/pb/pbdb.white.db - Permission denied;
Nov-18-10 13:07:00 PenaltyBox: cleaning WhiteBox finished; before=3, deleted=0;
Nov-18-10 13:07:00 Saving penalty records;
Nov-18-10 13:07:00 Error: orderedtie is unable to write to /var/db/assp/pb/pbdb.black.db - Permission denied;
Nov-18-10 13:07:00 Error: orderedtie is unable to write to /var/db/assp/pb/pbdb.white.db - Permission denied;

Fix: 

Added a chown assp:assp *.db to the /usr/local/etc/periodic/daily/510.assp
periodic script.
How-To-Repeat: Standard install of ASSP from the ports tree.
cd /var/db/assp/pb
chown assp:assp *
ls -als 

4 -rw-r--r--   1 assp  assp   112 Nov 18 14:46 pbdb.white.db
4 -rw-r--r--   1 assp  assp   112 Nov 18 14:34 pbdb.white.db.bak
4 -rw-r--r--   1 assp  assp   112 Nov 18 13:07 pbdb.white.db.tmp


/usr/local/etc/periodic/daily/510.assp
ls -als 

4 -rw-r--r--   1 root  assp   112 Nov 18 14:47 pbdb.white.db
4 -rw-r--r--   1 assp  assp   112 Nov 18 14:46 pbdb.white.db.bak
4 -rw-r--r--   1 assp  assp   112 Nov 18 13:07 pbdb.white.db.tmp
Comment 1 Olli Hauer freebsd_committer freebsd_triage 2011-01-05 08:41:33 UTC
Responsible Changed
From-To: freebsd-ports-bugs->ohauer

I'll take it.
Comment 2 dfilter service freebsd_committer freebsd_triage 2011-01-06 16:26:58 UTC
ohauer      2011-01-06 16:26:53 UTC

  FreeBSD ports repository

  Modified files:
    mail/assp            Makefile pkg-plist 
    mail/assp/files      periodic-assp.in periodic-assp.sh.in 
  Added files:
    mail/assp/files      pkg-install.in 
  Log:
   - fix periodic script(s) [1] [2]
   - use ports framework to create user/group
   - add pkg-install script to create links and directories
     so they are created during package install
   - use COPYTREE_SHARE instead of fix permissions and use INSTALL_DATA
   - fix order of pkg-plist
   - bump PORTREVISION
  
  PR:             [1] ports/152365 [2] ports/153350 (maintainer)
  Submitted by:   [1] Marco Walraven [2] Rusty Nejdl (maintainer)
  
  Revision  Changes    Path
  1.30      +15 -40    ports/mail/assp/Makefile
  1.2       +2 -2      ports/mail/assp/files/periodic-assp.in
  1.6       +2 -1      ports/mail/assp/files/periodic-assp.sh.in
  1.7       +20 -0     ports/mail/assp/files/pkg-install.in (new)
  1.14      +9 -9      ports/mail/assp/pkg-plist
_______________________________________________
cvs-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/cvs-all
To unsubscribe, send any mail to "cvs-all-unsubscribe@freebsd.org"
Comment 3 Olli Hauer freebsd_committer freebsd_triage 2011-01-06 17:05:40 UTC
State Changed
From-To: open->closed

Committed, 
Thanks!
Comment 4 Marco Walraven 2011-01-07 10:39:26 UTC
Hi,

Just saw that it was accepted. Tried patch yesterday and works
correctly. Thanks!

Regards,

Marco


On Tue, Dec 21, 2010 at 12:25:04PM -0600, Rusty Nejdl wrote:
> 
> 
> On Tue, 21 Dec 2010 12:19:27 -0600, Rusty Nejdl wrote: 
> 
> > Please
> find attached patch to correct for this. The last fix I submitted was
> incorrect and only fixed the permissions on some of the files while this
> one fixes it correctly. 
> > 
> > Thanks!
> > Rusty Nejdl
> 
> Sorry, that was a
> bad diff (old version). Please find the corrected one which I will be
> submitting for updating the port. 
> 
> Thanks!
> Rusty 


-- 
 Terantula - System & Network Integration - http://www.terantula.com
 Marco Walraven / phone:+31 64 3232 400 / mail:walraven@terantula.com
 pgp fingerprint: F2EE 122D 964C DE68 7380 6F95 3710 7719 E7EE 7A46
 pgpkey: E7EE7A46
 Location: Torenallee 20, Strijp-S, SFJ Building, 7th Floor, Room 7.02
 Postal: Postbus 2075 5600 CB Eindhoven, The Netherlands