system(3): Clarify return values
ClosedPublic
Actions

Authored by des on Tue, Feb 24, 1:44 PM.

Tags

None

Referenced Files

	F146419066: D55483.id172584.diff
	Mon, Mar 2, 1:19 PM

	Unknown Object (File)
	Sun, Mar 1, 2:24 PM

	Unknown Object (File)
	Sun, Mar 1, 7:11 AM

	Unknown Object (File)
	Sun, Mar 1, 3:33 AM

	Unknown Object (File)
	Sun, Mar 1, 2:25 AM

	Unknown Object (File)
	Fri, Feb 27, 7:10 AM

	Unknown Object (File)
	Fri, Feb 27, 12:11 AM

	Unknown Object (File)
	Thu, Feb 26, 4:57 AM

View All 14 Files

Subscribers

imp

ziaee

Details

Reviewers

bnovkov
kevans

Group Reviewers

Klara

Commits

rG7305604b29d3: system(3): Clarify return values

Summary

Our manual page currently states that system() will return 127 if it
fails to execute the shell. The actual return value is, to quote POSIX,
“as if the command language interpreter had terminated using exit(127)
or _exit(127)”.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

des created this revision.Tue, Feb 24, 1:44 PM

Herald added subscribers: ziaee, imp. · View Herald TranscriptTue, Feb 24, 1:44 PM

des requested review of this revision.Tue, Feb 24, 1:44 PM

Harbormaster completed remote builds in B70953: Diff 172584.Tue, Feb 24, 1:44 PM

des added a parent revision: D55482: system(3): Write our own tests.Tue, Feb 24, 1:44 PM

des added a child revision: D55484: system(3): Fix null case.

bnovkov accepted this revision as: bnovkov.Tue, Feb 24, 2:25 PM

This revision is now accepted and ready to land.Tue, Feb 24, 2:25 PM

kevans accepted this revision.Wed, Feb 25, 8:31 PM

Closed by commit rG7305604b29d3: system(3): Clarify return values (authored by des). · Explain WhyWed, Feb 25, 9:14 PM

This revision was automatically updated to reflect the committed changes.

des added a commit: rG7305604b29d3: system(3): Clarify return values.

Revision Contents
Changeset List

Path

Size

lib/

libc/

stdlib/

system.3

8 lines

Diff 172710

View Options

system(3): Clarify return valuesClosedPublicActions