Re: [PATCH] trace: Improve "trace show" command

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



----- Original Message -----
> On 03/23/2011 10:37 PM, Dave Anderson wrote:
> > So if you don't mind, I'll fix it like this:
> >
> >         buf[0] = 0;
> >         if ((file = popen(trace_cmd, "r"))) {
> >                 ret = fread(buf, 1, sizeof(buf), file);
> >                 buf[4095] = 0;
> >         }
> 
> It seems better:
> if (...) {
>         ret = fread(buf, 1, sizeof(buf) - 1, file);
>         buf[ret] = 0;
> }

Even better -- applied as above, and queued for crash-5.1.4.

Thanks Lai and Steve,
  Dave

--
Crash-utility mailing list
Crash-utility@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/crash-utility


[Index of Archives]     [Fedora Development]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]

 

Powered by Linux