cvs commit: src/sys/kern kern_prot.c src/sys/sys ucred.h

[Simon Schubert]

corecode    2007/01/08 13:32:57 PST

DragonFly src repository

  Modified files:
    sys/kern             kern_prot.c 
    sys/sys              ucred.h 
  Log:
  Fix the obscure at(1) permission problem.
  
  change_[er]uid() both use cratom(), however their consumers happened to cache
  the original cred in a variable.  If the cred happened to be shared before
  (for example due to an open(2)), a conditional lateron would still reference
  the old cred instead of the new one, which was instanciated in change_[er]uid.
  
  Fix this by returning the new cred from change_[er]uid and using this in
  subsequent conditionals.
  
  DragonFly-bug: http://bugs.dragonflybsd.org/issue509
  
  Revision  Changes    Path
  1.27      +11 -10    src/sys/kern/kern_prot.c
  1.9       +2 -2      src/sys/sys/ucred.h


http://www.dragonflybsd.org/cvsweb/src/sys/kern/kern_prot.c.diff?r1=1.26&r2=1.27&f=u
http://www.dragonflybsd.org/cvsweb/src/sys/sys/ucred.h.diff?r1=1.8&r2=1.9&f=u