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
* Mandatory Fields

Add a New Comment (Rich Markup)
   

Comment Type & Canned Response:
       No canned response available

 

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

 

There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

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.9