[Bug 2113] New: S_IWRITE is considered obsolete and should be replaced with S_IWUSR.

bugzilla-daemon at mindrot.org bugzilla-daemon at mindrot.org
Sun Jun 2 05:57:31 EST 2013


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

            Bug ID: 2113
           Summary: S_IWRITE is considered obsolete and should be replaced
                    with S_IWUSR.
           Product: Portable OpenSSH
           Version: 6.2p1
          Hardware: Other
                OS: Linux
            Status: NEW
          Severity: minor
          Priority: P5
         Component: Build system
          Assignee: unassigned-bugs at mindrot.org
          Reporter: nathan.osman at gmail.com

Created attachment 2287
  --> https://bugzilla.mindrot.org/attachment.cgi?id=2287&action=edit
Replaces instances of S_IWRITE with S_IWUSR.

The constant S_IWRITE is used in a couple of places in the OpenSSH
source code. According to the page below, this constant is obsolete:

 
http://www.gnu.org/savannah-checkouts/gnu/libc/manual/html_node/Permission-Bits.html

This causes the following errors when cross-compiling for the Android
platform:

  scp.c:1071:11: error: 'S_IWRITE' undeclared (first use in this
function)
  sftp-client.c:1054:13: error: 'S_IWRITE' undeclared (first use in
this function)

The equivalent constant that should be used in its place is 'S_IWUSR'.
I have attached a patch that corrects these instances.

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


More information about the openssh-bugs mailing list