openssh bugs - Aug 2020 - [Bug 3206] New: sftp client(32bit) chown command does not support uid >LONG_MAX

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

            Bug ID: 3206
           Summary: sftp client(32bit) chown command does not support uid
                    >LONG_MAX
           Product: Portable OpenSSH
           Version: 6.9p1
          Hardware: 68k
                OS: All
            Status: NEW
          Severity: normal
          Priority: P5
         Component: sftp
          Assignee: unassigned-bugs at mindrot.org
          Reporter: booking00 at sina.cn

Server could accept uid < ULONG_MAX. But client can only accept
uid<LONG_MAX. There is a gap between client and server. Variable l is
defiled as signed long. Is it possible to extend?

sftp> chown 2147483648 execute.sh
You must supply a numeric argument to the chown command.


        case I_CHOWN:
        case I_CHGRP:
                if ((optidx = parse_ch_flags(cmd, argv, argc, hflag))
== -1)
                        return -1;
                /* Get numeric arg (mandatory) */
                if (argc - optidx < 1)
                        goto need_num_arg;
                errno = 0;
                l = strtol(argv[optidx], &cp2, base);
                if (cp2 == argv[optidx] || *cp2 != '\0' ||
                    ((l == LONG_MIN || l == LONG_MAX) && errno =ERANGE)
||
                    l < 0) {
 need_num_arg:
                        error("You must supply a numeric argument "
                            "to the %s command.", cmd);
                        return -1;
                }

-- 
You are receiving this mail because:
You are watching the assignee of the bug.

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

Darren Tucker <dtucker at dtucker.net> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Blocks|                            |3217
                 CC|                            |dtucker at dtucker.net


Referenced Bugs:

https://bugzilla.mindrot.org/show_bug.cgi?id=3217
[Bug 3217] Tracking bug for 8.5 release
-- 
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.

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

--- Comment #1 from Darren Tucker <dtucker at dtucker.net> ---
Created attachment 3447
  --> https://bugzilla.mindrot.org/attachment.cgi?id=3447&action=edit
Use long long for chown uid

This seems to fix it.  On an i386 before:

sftp> chown 2147483648 /tmp/t
You must supply a numeric argument to the chown command.

and after:

sftp> chown 2147483648 /tmp/t
Changing owner on /tmp/t
sftp> exit

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

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

Darren Tucker <dtucker at dtucker.net> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
             Status|NEW                         |RESOLVED

--- Comment #2 from Darren Tucker <dtucker at dtucker.net> ---
Patch applied and will be in the next major release.  Thanks for the
report.

-- 
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.

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

Damien Miller <djm at mindrot.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |CLOSED

--- Comment #3 from Damien Miller <djm at mindrot.org> ---
close bugs that were resolved in OpenSSH 8.5 release cycle

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