Re: [PATCH V5 08/20] rtla: Helper functions for rtla
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
- Subject: Re: [PATCH V5 08/20] rtla: Helper functions for rtla
- From: Steven Rostedt <rostedt@xxxxxxxxxxx>
- Date: Tue, 26 Oct 2021 12:26:14 -0400
- Cc: Daniel Bristot de Oliveira <bristot@xxxxxxxxxx>, Ingo Molnar <mingo@xxxxxxxxxx>, Tom Zanussi <zanussi@xxxxxxxxxx>, Masami Hiramatsu <mhiramat@xxxxxxxxxx>, Juri Lelli <juri.lelli@xxxxxxxxxx>, Clark Williams <williams@xxxxxxxxxx>, John Kacur <jkacur@xxxxxxxxxx>, Peter Zijlstra <peterz@xxxxxxxxxxxxx>, Thomas Gleixner <tglx@xxxxxxxxxxxxx>, Sebastian Andrzej Siewior <bigeasy@xxxxxxxxxxxxx>, linux-rt-users@xxxxxxxxxxxxxxx, linux-trace-devel@xxxxxxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx
- In-reply-to: <YXgrLIiK8pwOVsBe@geo.homenetwork>
- References: <cover.1635181938.git.bristot@kernel.org> <bc01018cef80ddb95fd0ccf1ec72963be7e4fc19.1635181938.git.bristot@kernel.org> <YXgrLIiK8pwOVsBe@geo.homenetwork>
On Wed, 27 Oct 2021 00:22:04 +0800
Tao Zhou <tao.zhou@xxxxxxxxx> wrote:
> > + do {
> > + retval = read(in_fd, buffer, sizeof(buffer));
> > + if (read <= 0)
>
> check "retval" not read. Like:
>
> if (retval <= 0)
OUCH! Good check.
Daniel, did that not produce any warnings??
-- Steve
>
> > + goto out_close;
> > +
> > + retval = write(out_fd, buffer, retval);
> > + if (retval < 0)
> > + goto out_close;
> > + } while (retval > 0);
[Index of Archives]
[Linux USB Development]
[Linux USB Development]
[Linux Audio Users]
[Yosemite Hiking]
[Linux Kernel]
[Linux SCSI]