Re: oaf waitpid fix

[Maciej Stachowiak <mjs eazel com>]

Dan Winship <danw helixcode com> writes:

> I'm not sure why I suddenly need this (some gnome-vfs thing going on
> that I haven't figured out yet), but it's correct anyway.

Looks good, apply.

 - MAciej
 
> Index: ChangeLog
> ===================================================================
> RCS file: /cvs/gnome/oaf/ChangeLog,v
> retrieving revision 1.87
> diff -u -r1.87 ChangeLog
> --- ChangeLog	2000/09/15 08:47:06	1.87
> +++ ChangeLog	2000/09/17 18:30:31
> @@ -1,3 +1,8 @@
> +2000-09-17  Dan Winship  <danw helixcode com>
> +
> +	* liboaf/oaf-registration.c (oaf_server_by_forking): waitpid can
> +	return EINTR.
> +
>  2000-09-15  Jesus Bravo Alvarez  <jba pobox com>
>  
>  	* configure.in: Added Galician (gl) to ALL_LINGUAS
> Index: liboaf/oaf-registration.c
> ===================================================================
> RCS file: /cvs/gnome/oaf/liboaf/oaf-registration.c,v
> retrieving revision 1.17
> diff -u -r1.17 oaf-registration.c
> --- liboaf/oaf-registration.c	2000/07/26 01:01:50	1.17
> +++ liboaf/oaf-registration.c	2000/09/17 18:30:31
> @@ -304,7 +304,9 @@
>  	}
>  
>  	if (childpid) {
> -		waitpid (childpid, &status, 0);	/* de-zombify */
> +                /* de-zombify */
> +                while (waitpid (childpid, &status, 0) == -1 && errno == EINTR)
> +                        ;
>                  
>  		if (!WIFEXITED (status)) {
>  			OAF_GeneralError *errval;
> 
> _______________________________________________
> gnome-components-list mailing list
> gnome-components-list gnome org
> http://mail.gnome.org/mailman/listinfo/gnome-components-list