[patch] shutdown.2: POSIX compliant since version 3.7-rc1

[patch] shutdown.2: POSIX compliant since version 3.7-rc1

[saulery-GANU6spQydw]

From: Stéphane Aulery <saulery-GANU6spQydw@public.gmane.org>

Signed-off-by: Stéphane Aulery <saulery-GANU6spQydw@public.gmane.org>
---
 man2/shutdown.2 | 13 +++++--------
 1 file changed, 5 insertions(+), 8 deletions(-)

diff --git a/man2/shutdown.2 b/man2/shutdown.2
index ab39cfd..2174de3 100644
--- a/man2/shutdown.2
+++ b/man2/shutdown.2
@@ -102,14 +102,11 @@ respectively, and are defined in
 .I <sys/socket.h>
 since glibc-2.1.91.
 .SH BUGS
-As currently implemented,
-checks for the validity of
-.I how
-are done in domain-specific code, and not all domains perform these checks.
-.\" FIXME .
-.\" https://bugzilla.kernel.org/show_bug.cgi?id=47111
-Most notably, UNIX domain sockets simply ignore invalid values;
-this may change in the future.
+This implementation is POSIX compliant since version 3.7-rc1. Previously, UNIX
+domain and DECnet sockets simply ignored invalid values of
+.IR how ,
+due to a domain-specific code. Applications that used invalid values to
+circumvent this lack should be corrected.
 .SH SEE ALSO
 .BR connect (2),
 .BR socket (2),
-- 
2.1.4

--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [patch] shutdown.2: POSIX compliant since version 3.7-rc1

[Michael Kerrisk (man-pages)]

Hi Stéphane

These patches of yours are good, but lack info in commit messages
explaining why the change is being made, pointers bugzilla entries,
etc. Could you amend this patch and resend?

Thanks,

Michael
to

On 26 February 2015 at 11:38,  <saulery-GANU6spQydw@public.gmane.org> wrote:
> From: Stéphane Aulery <saulery-GANU6spQydw@public.gmane.org>
>
> Signed-off-by: Stéphane Aulery <saulery-GANU6spQydw@public.gmane.org>
> ---
>  man2/shutdown.2 | 13 +++++--------
>  1 file changed, 5 insertions(+), 8 deletions(-)
>
> diff --git a/man2/shutdown.2 b/man2/shutdown.2
> index ab39cfd..2174de3 100644
> --- a/man2/shutdown.2
> +++ b/man2/shutdown.2
> @@ -102,14 +102,11 @@ respectively, and are defined in
>  .I <sys/socket.h>
>  since glibc-2.1.91.
>  .SH BUGS
> -As currently implemented,
> -checks for the validity of
> -.I how
> -are done in domain-specific code, and not all domains perform these checks.
> -.\" FIXME .
> -.\" https://bugzilla.kernel.org/show_bug.cgi?id=47111
> -Most notably, UNIX domain sockets simply ignore invalid values;
> -this may change in the future.
> +This implementation is POSIX compliant since version 3.7-rc1. Previously, UNIX
> +domain and DECnet sockets simply ignored invalid values of
> +.IR how ,
> +due to a domain-specific code. Applications that used invalid values to
> +circumvent this lack should be corrected.
>  .SH SEE ALSO
>  .BR connect (2),
>  .BR socket (2),
> --
> 2.1.4
>



-- 
Michael Kerrisk
Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/
Linux/UNIX System Programming Training: http://man7.org/training/
--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [patch] shutdown.2: POSIX compliant since version 3.7-rc1

[Stéphane Aulery]

Le jeudi 26 février 2015 à 01:28:48, Michael Kerrisk (man-pages) a écrit :
> 
> These patches of yours are good, but lack info in commit messages
> explaining why the change is being made, pointers bugzilla entries,
> etc. Could you amend this patch and resend?

Yes, I will do that.

-- 
Stéphane Aulery
--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[patch] shutdown.2: POSIX compliant since version 3.7-rc1

[saulery-GANU6spQydw]

From: Stéphane Aulery <saulery-GANU6spQydw@public.gmane.org>

Synchronizing the manpage with the new POSIX compliant behavior of shutdown().
A warning message is kept to indicate possible problems with the old behavior
of UNIX domain sockets.

See bug report: https://bugzilla.kernel.org/show_bug.cgi?id=47111

Signed-off-by: Stéphane Aulery <saulery-GANU6spQydw@public.gmane.org>
---
 man2/shutdown.2 | 13 +++++--------
 1 file changed, 5 insertions(+), 8 deletions(-)

diff --git a/man2/shutdown.2 b/man2/shutdown.2
index ab39cfd..2174de3 100644
--- a/man2/shutdown.2
+++ b/man2/shutdown.2
@@ -102,14 +102,11 @@ respectively, and are defined in
 .I <sys/socket.h>
 since glibc-2.1.91.
 .SH BUGS
-As currently implemented,
-checks for the validity of
-.I how
-are done in domain-specific code, and not all domains perform these checks.
-.\" FIXME .
-.\" https://bugzilla.kernel.org/show_bug.cgi?id=47111
-Most notably, UNIX domain sockets simply ignore invalid values;
-this may change in the future.
+This implementation is POSIX compliant since version 3.7-rc1. Previously, UNIX
+domain and DECnet sockets simply ignored invalid values of
+.IR how ,
+due to a domain-specific code. Applications that used invalid values to
+circumvent this lack should be corrected.
 .SH SEE ALSO
 .BR connect (2),
 .BR socket (2),
-- 
2.1.4

--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [patch] shutdown.2: POSIX compliant since version 3.7-rc1

[Michael Kerrisk (man-pages)]

Hello Stéphane,

On 02/27/2015 01:21 AM, saulery-GANU6spQydw@public.gmane.org wrote:
> From: Stéphane Aulery <saulery-GANU6spQydw@public.gmane.org>
> 
> Synchronizing the manpage with the new POSIX compliant behavior of shutdown().
> A warning message is kept to indicate possible problems with the old behavior
> of UNIX domain sockets.
> 
> See bug report: https://bugzilla.kernel.org/show_bug.cgi?id=47111
>
> Signed-off-by: Stéphane Aulery <saulery-GANU6spQydw@public.gmane.org>
> ---
>  man2/shutdown.2 | 13 +++++--------
>  1 file changed, 5 insertions(+), 8 deletions(-)
> 
> diff --git a/man2/shutdown.2 b/man2/shutdown.2
> index ab39cfd..2174de3 100644
> --- a/man2/shutdown.2
> +++ b/man2/shutdown.2
> @@ -102,14 +102,11 @@ respectively, and are defined in
>  .I <sys/socket.h>
>  since glibc-2.1.91.
>  .SH BUGS
> -As currently implemented,
> -checks for the validity of
> -.I how
> -are done in domain-specific code, and not all domains perform these checks.
> -.\" FIXME .
> -.\" https://bugzilla.kernel.org/show_bug.cgi?id=47111
> -Most notably, UNIX domain sockets simply ignore invalid values;
> -this may change in the future.
> +This implementation is POSIX compliant since version 3.7-rc1. Previously, UNIX
> +domain and DECnet sockets simply ignored invalid values of
> +.IR how ,
> +due to a domain-specific code. Applications that used invalid values to
> +circumvent this lack should be corrected.
>  .SH SEE ALSO
>  .BR connect (2),
>  .BR socket (2),

I think the above patch probably goes into a little too much detail.
I think few people care about DECnet these days, so I'd prefer not
to clutter the page with mention of it.

I instead applied the patch below. Thanks for spotting that this kernel
bug had been fixed!

Cheers,

Michael

--- a/man2/shutdown.2
+++ b/man2/shutdown.2
@@ -102,14 +102,16 @@ respectively, and are defined in
 .I <sys/socket.h>
 since glibc-2.1.91.
 .SH BUGS
-As currently implemented,
-checks for the validity of
+Checks for the validity of
 .I how
-are done in domain-specific code, and not all domains perform these checks.
-.\" FIXME .
+are done in domain-specific code,
+and before Linux 3.7 not all domains performed these checks.
 .\" https://bugzilla.kernel.org/show_bug.cgi?id=47111
-Most notably, UNIX domain sockets simply ignore invalid values;
-this may change in the future.
+Most notably, UNIX domain sockets simply ignored invalid values.
+This problem was fixed for UNIX domain sockets
+.\" commit fc61b928dc4d72176cf4bd4d30bf1d22e599aefc
+.\" and for DECnet sockets in commit 46b66d7077b89fb4917ceef19b3f7dd86055c94a
+in Linux 3.7.
 .SH SEE ALSO
 .BR connect (2),
 .BR socket (2),

 


-- 
Michael Kerrisk
Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/
Linux/UNIX System Programming Training: http://man7.org/training/
--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [patch] shutdown.2: POSIX compliant since version 3.7-rc1

[Stéphane Aulery]

Hello Michael,

Le mardi 21 avril 2015 à 03:18:16, Michael Kerrisk (man-pages) a écrit :
> 
> On 02/27/2015 01:21 AM, saulery-GANU6spQydw@public.gmane.org wrote:
> > From: Stéphane Aulery <saulery-GANU6spQydw@public.gmane.org>
> > 
> > Synchronizing the manpage with the new POSIX compliant behavior of shutdown().
> > A warning message is kept to indicate possible problems with the old behavior
> > of UNIX domain sockets.
> > 
> > See bug report: https://bugzilla.kernel.org/show_bug.cgi?id=47111
> >
> > Signed-off-by: Stéphane Aulery <saulery-GANU6spQydw@public.gmane.org>
> > ---
> >  man2/shutdown.2 | 13 +++++--------
> >  1 file changed, 5 insertions(+), 8 deletions(-)
> > 
> > diff --git a/man2/shutdown.2 b/man2/shutdown.2
> > index ab39cfd..2174de3 100644
> > --- a/man2/shutdown.2
> > +++ b/man2/shutdown.2
> > @@ -102,14 +102,11 @@ respectively, and are defined in
> >  .I <sys/socket.h>
> >  since glibc-2.1.91.
> >  .SH BUGS
> > -As currently implemented,
> > -checks for the validity of
> > -.I how
> > -are done in domain-specific code, and not all domains perform these checks.
> > -.\" FIXME .
> > -.\" https://bugzilla.kernel.org/show_bug.cgi?id=47111
> > -Most notably, UNIX domain sockets simply ignore invalid values;
> > -this may change in the future.
> > +This implementation is POSIX compliant since version 3.7-rc1. Previously, UNIX
> > +domain and DECnet sockets simply ignored invalid values of
> > +.IR how ,
> > +due to a domain-specific code. Applications that used invalid values to
> > +circumvent this lack should be corrected.
> >  .SH SEE ALSO
> >  .BR connect (2),
> >  .BR socket (2),
> 
> I think the above patch probably goes into a little too much detail.
> I think few people care about DECnet these days, so I'd prefer not
> to clutter the page with mention of it.
> 
> I instead applied the patch below. Thanks for spotting that this kernel
> bug had been fixed!

Chic, even a bug in my list of conquests. Thank you Michael.

> Cheers,

-- 
Stéphane Aulery
--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[patch] shutdown.2: POSIX compliant since version 3.7-rc1

[saulery-GANU6spQydw]

From: Stéphane Aulery <saulery-GANU6spQydw@public.gmane.org>

Synchronizing the manpage with the new POSIX compliant behavior of shutdown().
A warning message is kept to indicate possible problems with the old behavior
of UNIX domain sockets.

See bug report: https://bugzilla.kernel.org/show_bug.cgi?id=47111

Signed-off-by: Stéphane Aulery <saulery-GANU6spQydw@public.gmane.org>
---
 man2/shutdown.2 | 13 +++++--------
 1 file changed, 5 insertions(+), 8 deletions(-)

diff --git a/man2/shutdown.2 b/man2/shutdown.2
index ab39cfd..2174de3 100644
--- a/man2/shutdown.2
+++ b/man2/shutdown.2
@@ -102,14 +102,11 @@ respectively, and are defined in
 .I <sys/socket.h>
 since glibc-2.1.91.
 .SH BUGS
-As currently implemented,
-checks for the validity of
-.I how
-are done in domain-specific code, and not all domains perform these checks.
-.\" FIXME .
-.\" https://bugzilla.kernel.org/show_bug.cgi?id=47111
-Most notably, UNIX domain sockets simply ignore invalid values;
-this may change in the future.
+This implementation is POSIX compliant since version 3.7-rc1. Previously, UNIX
+domain and DECnet sockets simply ignored invalid values of
+.IR how ,
+due to a domain-specific code. Applications that used invalid values to
+circumvent this lack should be corrected.
 .SH SEE ALSO
 .BR connect (2),
 .BR socket (2),
-- 
2.1.4

--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html