On 03-09-15 22:18, Lukasz K. wrote:
While the patch is designed to only fix the problem I was thinking if
it wouldn't be perhaps better to rename "context_0.uid_gid_set" to
something more function related like "context_0.uid_gid_chroot_set",
to make it more obvious for people who would look at the code in the
future that this member is actually meant for the function in general,
not only a certain part of it (UID/GID).
With above in mind I decided to introduce this change to the patch.
I also included few other "improvements" to make the function appear
more consistent and more obvious (like "c0->uid_gid_chroot_set = true"
separated and put at the end of the block).
At this point I'm leaving it up to you guys, this is just my another
suggestion, compiled & tested with latest git ([2.3_git
[git:master/291c227d2ccecaa9+]).
Thanks. This version looks good to me, but the patch won't apply. Most
likely your MUA messed up line wrapping. Could you resend the patch
using 'git send-email', or alternatively as an attachment?
-Steffan
