| Summary: | uid for expansion in ControlPath | ||||||
|---|---|---|---|---|---|---|---|
| Product: | Portable OpenSSH | Reporter: | Christian Hesse <mail> | ||||
| Component: | ssh | Assignee: | Assigned to nobody <unassigned-bugs> | ||||
| Status: | CLOSED FIXED | ||||||
| Severity: | enhancement | CC: | djm, dtucker | ||||
| Priority: | P5 | ||||||
| Version: | 7.0p1 | ||||||
| Hardware: | All | ||||||
| OS: | All | ||||||
| Bug Depends on: | |||||||
| Bug Blocks: | 2451 | ||||||
| Attachments: |
|
||||||
Comment on attachment 2687 [details]
uid for expansion in ControlPath
Looks fine to me
Comment on attachment 2687 [details] uid for expansion in ControlPath >- char thishost[NI_MAXHOST], shorthost[NI_MAXHOST], portstr[NI_MAXSERV]; >+ char thishost[NI_MAXHOST], shorthost[NI_MAXHOST], portstr[NI_MAXSERV], >+ uidstr[11]; > char cname[NI_MAXHOST]; I'd maybe make uidstr part of the following line for neatness. Also, someone will eventually make a 64 bit uid_t, so perhaps you want to spend the extra 10 bytes of stacks space now? applied (with uidstr sized to fit a 64-bit UID) Close all resolved bugs after 7.3p1 release |
Created attachment 2687 [details] uid for expansion in ControlPath Modern Linux systems create a private directory in /run/user/ for each user, named by numeric user id (uid). This adds a new character sequence '%i' for expansion in ControlPath to match this (and possibly other) directory.