On Fri, Sep 26, 2014 at 10:01:32PM +0200, Thomas H.P. Andersen wrote:
> From: Thomas Hindoe Paaboel Andersen <[email protected]>
> 
> Spotted with coverity. If parsing both /etc/os-release and
> /usr/lib/os-release fails then null would be passed on. The calls
> to parse the two files are allowed to fail. A empty /etc may not
> have had the /etc/os-release symlink restored yet and we just
> try again in the loop. If for whatever reason that does not happen
> then we now pass on 'n/a' instead of null.
> ---
>  src/bootchart/bootchart.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/bootchart/bootchart.c b/src/bootchart/bootchart.c
> index e127ad3..a359307 100644
> --- a/src/bootchart/bootchart.c
> +++ b/src/bootchart/bootchart.c
> @@ -477,7 +477,7 @@ int main(int argc, char *argv[]) {
>                  exit (EXIT_FAILURE);
>          }
>  
> -        svg_do(build);
> +        svg_do(strna(build));
>  
I think think one is valid, but got forgotten. Applied now.

Zbyszek
_______________________________________________
systemd-devel mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/systemd-devel

Reply via email to