bugConcurrent Versions System - Bugs: bug #19856, cvs compiled with pam loose it's...

 
 

bug #19856: cvs compiled with pam loose it's nice (priority) value.

Submitted by:  None
Submitted on:  Fri 11 May 2007 02:06:20 PM UTC  
 
Category:  Bug Fix (patch attached) Severity:  3 - Normal
Item Group:  None Status:  None
Privacy:  Public Assigned to:  None
Open/Closed:  Open Release: 
Fixed Release:  None Fixed Feature Release:  None

Add a New Comment(Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

Fri 11 May 2007 02:06:20 PM UTC, original submission:

Hello,

I am using cvs 1.12.13.1 in mode pserver, compiled with pam because my system authenticate on ldap.
For some clients, i run cvs with a high nice value (lower priority).

I have noticed that cvs "forget" it's nice value after launching.
after debugging i have found that it happens in :
file server.c, function switch_to_user() the call to
pam_open_session() reset the nice value !

i have tested with 1.12.13 and 1.12.13.1, i guess the problem does not happen with 1.11.x since it is not pam aware !

I don't know pam, maybe it is possible to prevent this behavior,
but i have find that getting the priority before the call to
pam_open_session() and re-setting it after, fix the problem.

you will find a patch for server.c attached.

Best Regards

Anonymous

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

Attached Files
file #12749:  server.c.patch added by None (594B - text/x-patch - patch to server.c, to remember nice value)

 

Depends on the following items: None found

Items that depend on this one: None found

 

CC list is empty

 

Do you think this task is very important?
If so, you can add your encouragement to it.
This task has 0 encouragements so far.

Only logged-in users can vote.

 

 

 

Follows 1 latest change.

Date Changed by Updated Field Previous Value => Replaced by
2007-05-11 None Attached File- => Added server.c.patch, #12749

Back to the top


Powered by Savane 3.3