Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Wrong parameters passed to the Registry inside XrdSecsssID::Register() method #689

Closed
mpatrascoiu opened this issue Apr 17, 2018 · 0 comments

Comments

@mpatrascoiu
Copy link
Contributor

In the XrdSecsssID::Register method we have this variable:
int hOpt = (doRep ? Hash_replace : Hash_default) | Hash_dofree;

and the following "Registry.Add" call
rc = (Registry.Add(lid, idP, hOpt) ? 0 : 1);

However, this assigns the options variable (hOpt) to the LifeTime parameter of the Registry.Add function.

In order to pass the options and keep the default LifeTime, the call should be:

rc = (Registry.Add(lid, idP, 0, (XrdOucHash_Options) hOpt) ? 0 : 1);

mpatrascoiu added a commit to mpatrascoiu/xrootd that referenced this issue Jun 7, 2018
mpatrascoiu added a commit to mpatrascoiu/xrootd that referenced this issue Jun 8, 2018
mpatrascoiu added a commit to mpatrascoiu/xrootd that referenced this issue Jun 11, 2018
@abh3 abh3 closed this as completed in 6aa690d Aug 21, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant