All of lore.kernel.org
 help / color / mirror / Atom feed
* Patch to fix newrole/more problem where more fails because terminal is not open correctly
@ 2006-12-11 20:40 Daniel J Walsh
  2006-12-11 22:00 ` Karl MacMillan
  0 siblings, 1 reply; 2+ messages in thread
From: Daniel J Walsh @ 2006-12-11 20:40 UTC (permalink / raw)
  To: SE Linux, Stephen Smalley

diff --exclude-from=exclude --exclude POTFILES.in --exclude='*.po' 
--exclude='*.pot' -N -u -r nsapolicycoreutils/newrole/newrole.c 
policycoreutils-1.33.6/newrole/newrole.c
--- nsapolicycoreutils/newrole/newrole.c        2006-11-29 
17:11:18.000000000 -0500
+++ policycoreutils-1.33.6/newrole/newrole.c    2006-12-08 
10:34:48.000000000 -0500
@@ -1120,10 +1120,10 @@
        fd = open(ttyn, O_RDONLY);
        if (fd != 0)
                goto err_close_pam;
-       fd = open(ttyn, O_WRONLY);
+       fd = open(ttyn, O_RDWR);
        if (fd != 1)
                goto err_close_pam;
-       fd = open(ttyn, O_WRONLY);
+       fd = open(ttyn, O_RDWR);
        if (fd != 2)
                goto err_close_pam;


--
This message was distributed to subscribers of the selinux mailing list.
If you no longer wish to subscribe, send mail to majordomo@tycho.nsa.gov with
the words "unsubscribe selinux" without quotes as the message.

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: Patch to fix newrole/more problem where more fails because terminal is not open correctly
  2006-12-11 20:40 Patch to fix newrole/more problem where more fails because terminal is not open correctly Daniel J Walsh
@ 2006-12-11 22:00 ` Karl MacMillan
  0 siblings, 0 replies; 2+ messages in thread
From: Karl MacMillan @ 2006-12-11 22:00 UTC (permalink / raw)
  To: Daniel J Walsh; +Cc: SE Linux, Stephen Smalley

Daniel J Walsh wrote:
> diff --exclude-from=exclude --exclude POTFILES.in --exclude='*.po' 
> --exclude='*.pot' -N -u -r nsapolicycoreutils/newrole/newrole.c 
> policycoreutils-1.33.6/newrole/newrole.c
> --- nsapolicycoreutils/newrole/newrole.c        2006-11-29 
> 17:11:18.000000000 -0500
> +++ policycoreutils-1.33.6/newrole/newrole.c    2006-12-08 
> 10:34:48.000000000 -0500
> @@ -1120,10 +1120,10 @@
>        fd = open(ttyn, O_RDONLY);
>        if (fd != 0)
>                goto err_close_pam;
> -       fd = open(ttyn, O_WRONLY);
> +       fd = open(ttyn, O_RDWR);
>        if (fd != 1)
>                goto err_close_pam;
> -       fd = open(ttyn, O_WRONLY);
> +       fd = open(ttyn, O_RDWR);
>        if (fd != 2)
>                goto err_close_pam;
> 

Acked-by: Karl MacMillan <kmacmillan@mentalrootkit.com>




--
This message was distributed to subscribers of the selinux mailing list.
If you no longer wish to subscribe, send mail to majordomo@tycho.nsa.gov with
the words "unsubscribe selinux" without quotes as the message.

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2006-12-11 22:00 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2006-12-11 20:40 Patch to fix newrole/more problem where more fails because terminal is not open correctly Daniel J Walsh
2006-12-11 22:00 ` Karl MacMillan

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.