diff options
| author | Zdenek Kabelac <zkabelac@redhat.com> | 2021-09-20 10:31:45 +0200 |
|---|---|---|
| committer | Zdenek Kabelac <zkabelac@redhat.com> | 2021-09-20 14:26:09 +0200 |
| commit | 055406bb398ddf60f18c545baffcec4ee4d5646e (patch) | |
| tree | 70b5aa5b161e92dbcdd4d32e7802db4fd5d523ab /test/lib | |
| parent | 9721f3e0ff6fd841b62b6bcccceee2ebb914e96b (diff) | |
| download | lvm2-055406bb398ddf60f18c545baffcec4ee4d5646e.tar.gz | |
cov: validate syscall result
Diffstat (limited to 'test/lib')
| -rw-r--r-- | test/lib/brick-shelltest.h | 3 | ||||
| -rw-r--r-- | test/lib/not.c | 6 |
2 files changed, 7 insertions, 2 deletions
diff --git a/test/lib/brick-shelltest.h b/test/lib/brick-shelltest.h index 0b1f9dfee..6bc02e0d2 100644 --- a/test/lib/brick-shelltest.h +++ b/test/lib/brick-shelltest.h @@ -424,7 +424,8 @@ struct FdSink : Sink { { TimedBuffer::Line line = stream.shift( force ); std::string out = fmt.format( line ); - write( fd, out.c_str(), out.length() ); + if ( write( fd, out.c_str(), out.length() ) < (int)out.length() ) + perror( "short write" ); } virtual void sync( bool force ) { diff --git a/test/lib/not.c b/test/lib/not.c index a5c52a8e6..0cff13062 100644 --- a/test/lib/not.c +++ b/test/lib/not.c @@ -86,7 +86,11 @@ int main(int args, char **argv) { /* should not be accessible */ return FAILURE; } else { /* parent */ - waitpid(pid, &status, 0); + if (waitpid(pid, &status, 0) < 0) { + fprintf(stderr, "Process %d failed on waitpid.\n", pid); + return FAILURE; + } + if (!WIFEXITED(status)) { if (WIFSIGNALED(status)) fprintf(stderr, |