[Bug libc/18675] New: fpathconf(_PC_NAME_MAX) fails against large filesystems for 32bit processes

[Bug libc/18675] New: fpathconf(_PC_NAME_MAX) fails against large filesystems for 32bit processes

[jmaggard at netgear dot com]

https://sourceware.org/bugzilla/show_bug.cgi?id=18675

            Bug ID: 18675
           Summary: fpathconf(_PC_NAME_MAX) fails against large
                    filesystems for 32bit processes
           Product: glibc
           Version: 2.21
            Status: NEW
          Severity: normal
          Priority: P2
         Component: libc
          Assignee: unassigned at sourceware dot org
          Reporter: jmaggard at netgear dot com
                CC: drepper.fsp at gmail dot com
  Target Milestone: ---

Created attachment 8439
  --> https://sourceware.org/bugzilla/attachment.cgi?id=8439&action=edit
Proposed fix

Basically identical to bug #15514.

fpathconf(_PC_NAME_MAX) is implemented on top of fstatfs().  The 32bit version
therefore fails with the error EOVERFLOW if the filesystem that the path is on
is so large that its block count cannot be returned by the non-LFS version of
fstatfs().

Calls to fstatfs() that are completely internal to libc, such as this one,
should use the LFS version, __fstatvfs64().

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug libc/18675] fpathconf(_PC_NAME_MAX) fails against large filesystems for 32bit processes

[ppluzhnikov at google dot com]

https://sourceware.org/bugzilla/show_bug.cgi?id=18675

Paul Pluzhnikov <ppluzhnikov at google dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |ppluzhnikov at google dot com
           Assignee|unassigned at sourceware dot org   |ppluzhnikov at google dot com

--- Comment #1 from Paul Pluzhnikov <ppluzhnikov at google dot com> ---
The patch looks good to me.

Can you mail to libc-alpha@sourceware.org for approval?

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug libc/18675] fpathconf(_PC_NAME_MAX) fails against large filesystems for 32bit processes

[ppluzhnikov at google dot com]

https://sourceware.org/bugzilla/show_bug.cgi?id=18675

Paul Pluzhnikov <ppluzhnikov at google dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED

--- Comment #2 from Paul Pluzhnikov <ppluzhnikov at google dot com> ---
Mailed the patch:  https://sourceware.org/ml/libc-alpha/2015-09/msg00170.html

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug libc/18675] fpathconf(_PC_NAME_MAX) fails against large filesystems for 32bit processes

[cvs-commit at gcc dot gnu.org]

https://sourceware.org/bugzilla/show_bug.cgi?id=18675

--- Comment #3 from cvs-commit at gcc dot gnu.org <cvs-commit at gcc dot gnu.org> ---
This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "GNU C Library master sources".

The branch, master has been updated
       via  d18c36e6007b03533a38c890c68544daa78d301a (commit)
      from  ec999b8e5ede67f42759657beb8c5fef87c8cc63 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
https://sourceware.org/git/gitweb.cgi?p=glibc.git;h=d18c36e6007b03533a38c890c68544daa78d301a

commit d18c36e6007b03533a38c890c68544daa78d301a
Author: Paul Pluzhnikov <ppluzhnikov@google.com>
Date:   Wed Sep 9 18:41:25 2015 -0700

    To fix BZ #18675, use __fstatvfs64 in __fpathconf.

-----------------------------------------------------------------------

Summary of changes:
 ChangeLog                 |    5 +++++
 NEWS                      |    6 +++---
 sysdeps/posix/fpathconf.c |   14 +++-----------
 3 files changed, 11 insertions(+), 14 deletions(-)

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug libc/18675] fpathconf(_PC_NAME_MAX) fails against large filesystems for 32bit processes

[ppluzhnikov at google dot com]

https://sourceware.org/bugzilla/show_bug.cgi?id=18675

Paul Pluzhnikov <ppluzhnikov at google dot com> changed:

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

--- Comment #4 from Paul Pluzhnikov <ppluzhnikov at google dot com> ---
Fixed on trunk.

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug libc/18675] fpathconf(_PC_NAME_MAX) fails against large filesystems for 32bit processes

[jmaggard at netgear dot com]

https://sourceware.org/bugzilla/show_bug.cgi?id=18675

--- Comment #5 from Justin Maggard <jmaggard at netgear dot com> ---
Heh, sorry, my Bugzilla email filter gobbled up your responses here.  Thanks
for taking care of that!

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug libc/18675] fpathconf(_PC_NAME_MAX) fails against large filesystems for 32bit processes

[jsm28 at gcc dot gnu.org]

https://sourceware.org/bugzilla/show_bug.cgi?id=18675

Joseph Myers <jsm28 at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|---                         |2.23

-- 
You are receiving this mail because:
You are on the CC list for the bug.