[Bug nis/22701] New: Incomplete removal of libnsl

classic Classic list List threaded Threaded
11 messages Options
Reply | Threaded
Open this post in threaded view
|

[Bug nis/22701] New: Incomplete removal of libnsl

tromey at sourceware dot org
https://sourceware.org/bugzilla/show_bug.cgi?id=22701

            Bug ID: 22701
           Summary: Incomplete removal of libnsl
           Product: glibc
           Version: 2.26
            Status: NEW
          Severity: minor
          Priority: P3
         Component: nis
          Assignee: unassigned at sourceware dot org
          Reporter: fweimer at redhat dot com
                CC: kukuk at suse dot de
  Target Milestone: ---
             Flags: security-

libnsl still defines some non-compat symbols:

xdr_ypall@@GLIBC_2.2.5
__nis_default_ttl@@GLIBC_2.2.5
yp_maplist@@GLIBC_2.2.5

Furthermore, the development symbolic link is still installed by “make
install”:

/usr/lib64/libnsl.so

--
You are receiving this mail because:
You are on the CC list for the bug.
Reply | Threaded
Open this post in threaded view
|

[Bug nis/22701] Incomplete removal of libnsl

tromey at sourceware dot org
https://sourceware.org/bugzilla/show_bug.cgi?id=22701

--- Comment #1 from Florian Weimer <fweimer at redhat dot com> ---
Also:

/usr/lib64/libnsl.a

--
You are receiving this mail because:
You are on the CC list for the bug.
Reply | Threaded
Open this post in threaded view
|

[Bug nis/22701] Incomplete removal of libnsl

tromey at sourceware dot org
In reply to this post by tromey at sourceware dot org
https://sourceware.org/bugzilla/show_bug.cgi?id=22701

Florian Weimer <fweimer at redhat dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
           Assignee|unassigned at sourceware dot org   |fweimer at redhat dot com

--- Comment #2 from Florian Weimer <fweimer at redhat dot com> ---
Patches posted:

https://sourceware.org/ml/libc-alpha/2018-01/msg00389.html
https://sourceware.org/ml/libc-alpha/2018-01/msg00404.html

--
You are receiving this mail because:
You are on the CC list for the bug.
Reply | Threaded
Open this post in threaded view
|

[Bug nis/22701] Incomplete removal of libnsl

tromey at sourceware dot org
In reply to this post by tromey at sourceware dot org
https://sourceware.org/bugzilla/show_bug.cgi?id=22701

--- 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  9a08a366a7e7ddffe62113a9ffe5e50605ea0924 (commit)
      from  f172187b2d5bd9d22a5311b3ecd89d4ce5dbd2da (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=9a08a366a7e7ddffe62113a9ffe5e50605ea0924

commit 9a08a366a7e7ddffe62113a9ffe5e50605ea0924
Author: Florian Weimer <[hidden email]>
Date:   Fri Jan 12 08:42:56 2018 +0100

    libnsl: Do not install libnsl.so, libnsl.a if NIS is disabled [BZ #22701]

    This also skips building the .o files for libnsl.a.

    Reviewed-by: Carlos O'Donell <[hidden email]>

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

Summary of changes:
 ChangeLog    |    7 +++++++
 nis/Makefile |   10 +++++++++-
 2 files changed, 16 insertions(+), 1 deletions(-)

--
You are receiving this mail because:
You are on the CC list for the bug.
Reply | Threaded
Open this post in threaded view
|

[Bug nis/22701] Incomplete removal of libnsl

tromey at sourceware dot org
In reply to this post by tromey at sourceware dot org
https://sourceware.org/bugzilla/show_bug.cgi?id=22701

--- Comment #4 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  5503f413bf4221d26f83333bb42d7b2cd056236c (commit)
      from  caaf7804e26c1a4f28a1bf7cfe1075d06bce1255 (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=5503f413bf4221d26f83333bb42d7b2cd056236c

commit 5503f413bf4221d26f83333bb42d7b2cd056236c
Author: Andreas Schwab <[hidden email]>
Date:   Fri Jan 26 19:01:03 2018 +0100

    Build only shared libnsl objects if NIS is disabled (bug 22701)

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

Summary of changes:
 ChangeLog    |    6 ++++++
 nis/Makefile |    3 ++-
 2 files changed, 8 insertions(+), 1 deletions(-)

--
You are receiving this mail because:
You are on the CC list for the bug.
Reply | Threaded
Open this post in threaded view
|

Re: [Bug nis/22701] Incomplete removal of libnsl

Joseph Myers
Do the commits here completely resolve this bug (in which case it should
be marked as FIXED with target milestone set) or are there still issues
left to fix?

--
Joseph S. Myers
[hidden email]
Reply | Threaded
Open this post in threaded view
|

[Bug nis/22701] Incomplete removal of libnsl

tromey at sourceware dot org
In reply to this post by tromey at sourceware dot org
https://sourceware.org/bugzilla/show_bug.cgi?id=22701

--- Comment #5 from joseph at codesourcery dot com <joseph at codesourcery dot com> ---
Do the commits here completely resolve this bug (in which case it should
be marked as FIXED with target milestone set) or are there still issues
left to fix?

--
You are receiving this mail because:
You are on the CC list for the bug.
Reply | Threaded
Open this post in threaded view
|

[Bug nis/22701] Incomplete removal of libnsl

tromey at sourceware dot org
In reply to this post by tromey at sourceware dot org
https://sourceware.org/bugzilla/show_bug.cgi?id=22701

--- Comment #6 from Florian Weimer <fweimer at redhat dot com> ---
(In reply to [hidden email] from comment #5)
> Do the commits here completely resolve this bug (in which case it should
> be marked as FIXED with target milestone set) or are there still issues
> left to fix?

No, this patch still needs review:

https://sourceware.org/ml/libc-alpha/2018-01/msg00389.html

But with the already committed patch (no .so symlink and no static library),
this is mostly a cosmetic issue.

--
You are receiving this mail because:
You are on the CC list for the bug.
Reply | Threaded
Open this post in threaded view
|

[Bug nis/22701] Incomplete removal of libnsl

tromey at sourceware dot org
In reply to this post by tromey at sourceware dot org
https://sourceware.org/bugzilla/show_bug.cgi?id=22701

Florian Weimer <fweimer at redhat dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|---                         |FIXED
   Target Milestone|---                         |2.27

--- Comment #7 from Florian Weimer <fweimer at redhat dot com> ---
Fixed for 2.27.

--
You are receiving this mail because:
You are on the CC list for the bug.
Reply | Threaded
Open this post in threaded view
|

[Bug nis/22701] Incomplete removal of libnsl

tromey at sourceware dot org
In reply to this post by tromey at sourceware dot org
https://sourceware.org/bugzilla/show_bug.cgi?id=22701

--- Comment #8 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  cdd14619a713ab41e26ba700add4880604324dbb (commit)
      from  b7ccb5de359e8dd9918dbedfc1f4d96e06fcd634 (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=cdd14619a713ab41e26ba700add4880604324dbb

commit cdd14619a713ab41e26ba700add4880604324dbb
Author: Florian Weimer <[hidden email]>
Date:   Mon Jan 29 17:00:14 2018 +0100

    libnsl: Turn remaining symbols into compat symbols [BZ #22701]

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

Summary of changes:
 ChangeLog               |   17 +++++++++++++++++
 include/rpcsvc/nislib.h |    1 +
 include/rpcsvc/yp.h     |    4 ++++
 include/rpcsvc/ypclnt.h |    4 ++++
 nis/Makefile            |    3 ++-
 nis/nis_defaults.c      |    1 +
 nis/rpcsvc/ypclnt.h     |    3 ---
 nis/yp_xdr.c            |    2 +-
 nis/ypclnt.c            |    1 +
 9 files changed, 31 insertions(+), 5 deletions(-)

--
You are receiving this mail because:
You are on the CC list for the bug.
Reply | Threaded
Open this post in threaded view
|

[Bug nis/22701] Incomplete removal of libnsl

tromey at sourceware dot org
In reply to this post by tromey at sourceware dot org
https://sourceware.org/bugzilla/show_bug.cgi?id=22701

Jeremi <jeremip11 at gmail dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |jeremip11 at gmail dot com

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