Fwd: kprocess - close all open files in the child

[Waldo Bastian]

I have reverted the patch for KDE 3.1 since it breaks backwards compatibility 
and the error handling of KProcess itself. Will fix sycoca to set the close 
on exec flag on its fd.

Cheers,
Waldo

On Wednesday 18 December 2002 23:53, Carsten Pfeiffer wrote:
> -----BEGIN PGP SIGNED MESSAGE-----
>
> - ----------  Forwarded Message  ----------
>
> Subject: kprocess - close all open files in the child
> Date: Wednesday 18 December 2002 09:17
> From: Klaus-Dieter Krannich <kd at Math.TU-Cottbus.DE>
> To: kde-devel at kde.org
>
> Hi all,
>
> Sometimes it is necessary to have additonal (beside stdin/stdout/stderr)
> communication channels between the caller and the child. In traditional
> GUI programming this is often done by opening a pipe/socketpair in the
> caller and passing a filedescriptor via commandline args to the child.
> The recently added 'close all open files in the child after the fork'
> feature in kprocess makes such an approach impossible. It would be
> really nice, if this behaviour was the default, but could be turned off
> by an appropriate flag at construction time.
>
> BTW, "Xterm started from minicli really doesn't need to have the ksycoca
> file open" is a weak argument w.r.t. the 'brute force' loop over all
> possible fd's. Wouldn't it be better to fix minicli ? :-)
>
> Greetings
>
> 	K-D
>
> >> Visit http://mail.kde.org/mailman/listinfo/kde-devel#unsub to
> >> unsubscribe <<
>
> - -------------------------------------------------------
>
> - --
> Carsten Pfeiffer
> -----BEGIN PGP SIGNATURE-----
>
> iQEVAwUBPgD8U6WgYMJuwmZtAQGsXQf/dHQ/ulvCXlS9VfdflDffXfeqUIzVWLUU
> 1Q3tABC1LnXt/LI/8MqxD3snMjgF5BTZrxhFSfBqcw7mEQ4mpdP+8GpK7p4PaHIy
> KYrcG7LgQrimsyJDPPUqhr39nhbWM0pjvXjsyqSpOFow2YJzyNibC2x9u1voehQi
> 0pMxHbt2MHdBkuVNJhHhiGU4ljtstwABwAk/3qFbKr1vvYQHME4WTNXLo8qFG2xw
> qwqzZqdS9y2jtVggjH7QnPrcRa6G6ArpzXcKECElEVYnTRywzw1jP29GdgfV4OWw
> Nn8N7USvt1/tXK2rsAZNTMuFC33f+bPY+bBH+nVaXp2hkJXBRALWrw==
> =kmgT
> -----END PGP SIGNATURE-----

-- 
bastian at kde.org -=|[ SuSE, The Linux Desktop Experts ]|=- bastian at suse.com