[Bug 2870] Support "%i" token expansion in the IdentityAgent option

[bugzilla-daemon at bugzilla.mindrot.org]

https://bugzilla.mindrot.org/show_bug.cgi?id=2870

Darren Tucker <dtucker at dtucker.net> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Attachment #3156|ok?(dtucker at dtucker.net)    |ok+
              Flags|                            |

--- Comment #4 from Darren Tucker <dtucker at dtucker.net> ---
Comment on attachment 3156
  --> https://bugzilla.mindrot.org/attachment.cgi?id=3156
Cast to safe integral value

>+	snprintf(uidstr, sizeof(uidstr), "%llu",
>+	    (unsigned long long)pw->pw_uid);

check for truncation (or use xasprintf)?  shouldn't happen with 64bits,
but maybe someone will make them GUIDs or something one day.

otherwise ok.

-- 
You are receiving this mail because:
You are watching someone on the CC list of the bug.
You are watching the assignee of the bug.