samba provisioning - uidNumber not growing

I had a problem with uidNumber when creating new users. The uidNumber of the new user is the same as the last created user. So when I created bunch of users they all had the same uidNumber. Which obviously not good. Never done php before, but this rough patch works for me:

diff ~before/ldapprov_samba.module ~after/ldapprov_samba.module
116c116
<       $uid_number = array_pop($uid_numbers);
---
>       $uid_number = array_pop($uid_numbers) + 1;