[FOSS_TLK]kernel: boot: Fix coverity hits
Aaron Gamble [Tue, 3 Feb 2015 19:17:43 +0000 (11:17 -0800)]
Prevent strlen(NULL)
Coverity id : 28173

Correct boot_params_ptr->param_string check
Coverity id : 28121

Change-Id: Ie99a5cbb030fcab560e88b4fadb1f6cf7a81f1b6
Reviewed-on: http://git-master/r/715779
Reviewed-by: Automatic_Commit_Validation_User
Reviewed-by: Varun Wadekar <vwadekar@nvidia.com>
Tested-by: Varun Wadekar <vwadekar@nvidia.com>

kernel/boot.c

index 3fa7826..7b7423a 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2012-2013, NVIDIA CORPORATION. All rights reserved
+ * Copyright (c) 2012-2015, NVIDIA CORPORATION. All rights reserved
  *
  * Permission is hereby granted, free of charge, to any person obtaining
  * a copy of this software and associated documentation files
@@ -47,11 +47,13 @@ struct cmdline_member {
 static int getoffsetof(char *source, char c)
 {
        int offset = 0;
-       int len = strlen(source);
+       int len;
 
        if (!source)
                return 0;
 
+       len = strlen(source);
+
        while (*source++ != c && len-- > 0)
                offset++;
 
@@ -76,7 +78,7 @@ void parse_cmdline(void)
        };
 
        if (!boot_params_ptr ||
-           !boot_params_ptr->param_string)
+           !boot_params_ptr->param_string[0])
                return;
 
        cmdline_len = boot_params_ptr->param_string_sz;