Re: [Pam-developers] [PATCH] pam_succed_if cmp function will get error when parameter > 2^31
On Po, 2014-09-29 at 11:28 +0800, Jianhai Luan wrote:
Hi all,
When I try to pam_success_if to author, I happen the below issue:
“ httpd: pam_successed_if(https:auth): requirement “uid >=100” not met by user
“sarah_anderson”"
And I ensure that the uid should be larger than 100 by the below command:
#getent passwd sarah_anderson
sarah_anderson:x:2300004883:18010:Sarah
I do the attachment patch to fix the issue. Please review and give me some advice about
the patch.
Another possibility would be to use long instead of unsigned int type.
It is debatable which one is more correct though. For future
extensibility (if we add some field that could get negative values), I
would prefer using long.
--
Tomas Mraz
No matter how far down the wrong road you've gone, turn back.
Turkish proverb
(You'll never know whether the road is wrong though.)