gpu: nvgpu: MISRA 21.6 fixes in hal netlist

Use strcpy/strcat instead of sprintf to fix MISRA 21.6
violations.

JIRA NVGPU-3420

Change-Id: I70314bc9b407370961bd46434bb355ebb8d1df7d
Signed-off-by: Seshendra Gadagottu <sgadagottu@nvidia.com>
Reviewed-on: https://git-master.nvidia.com/r/2115925
Reviewed-by: svc-mobile-coverity <svc-mobile-coverity@nvidia.com>
Reviewed-by: svc-mobile-misra <svc-mobile-misra@nvidia.com>
GVS: Gerrit_Virtual_Submit
Reviewed-by: Vinod Gopalakrishnakurup <vinodg@nvidia.com>
Reviewed-by: mobile promotions <svcmobile_promotions@nvidia.com>
Tested-by: mobile promotions <svcmobile_promotions@nvidia.com>
This commit is contained in:
Seshendra Gadagottu
2019-05-09 14:30:33 -07:00
committed by mobile promotions
parent 62a7fde536
commit 76aac6b183
5 changed files with 19 additions and 19 deletions

View File

@@ -31,27 +31,27 @@ int gm20b_netlist_get_name(struct gk20a *g, int index, char *name)
switch (index) { switch (index) {
#ifdef GM20B_NETLIST_IMAGE_FW_NAME #ifdef GM20B_NETLIST_IMAGE_FW_NAME
case NETLIST_FINAL: case NETLIST_FINAL:
(void) sprintf(name, GM20B_NETLIST_IMAGE_FW_NAME); (void) strcpy(name, GM20B_NETLIST_IMAGE_FW_NAME);
break; break;
#endif #endif
#ifdef GK20A_NETLIST_IMAGE_A #ifdef GK20A_NETLIST_IMAGE_A
case NETLIST_SLOT_A: case NETLIST_SLOT_A:
(void) sprintf(name, NVGPU_NETLIST_IMAGE_A); (void) strcpy(name, NVGPU_NETLIST_IMAGE_A);
break; break;
#endif #endif
#ifdef NVGPU_NETLIST_IMAGE_B #ifdef NVGPU_NETLIST_IMAGE_B
case NETLIST_SLOT_B: case NETLIST_SLOT_B:
(void) sprintf(name, NVGPU_NETLIST_IMAGE_B); (void) strcpy(name, NVGPU_NETLIST_IMAGE_B);
break; break;
#endif #endif
#ifdef NVGPU_NETLIST_IMAGE_C #ifdef NVGPU_NETLIST_IMAGE_C
case NETLIST_SLOT_C: case NETLIST_SLOT_C:
(void) sprintf(name, NVGPU_NETLIST_IMAGE_C); (void) strcpy(name, NVGPU_NETLIST_IMAGE_C);
break; break;
#endif #endif
#ifdef NVGPU_NETLIST_IMAGE_D #ifdef NVGPU_NETLIST_IMAGE_D
case NETLIST_SLOT_D: case NETLIST_SLOT_D:
(void) sprintf(name, NVGPU_NETLIST_IMAGE_D); (void) strcpy(name, NVGPU_NETLIST_IMAGE_D);
break; break;
#endif #endif
default: default:

View File

@@ -31,27 +31,27 @@ int gp10b_netlist_get_name(struct gk20a *g, int index, char *name)
switch (index) { switch (index) {
#ifdef GP10B_NETLIST_IMAGE_FW_NAME #ifdef GP10B_NETLIST_IMAGE_FW_NAME
case NETLIST_FINAL: case NETLIST_FINAL:
(void) sprintf(name, GP10B_NETLIST_IMAGE_FW_NAME); (void) strcpy(name, GP10B_NETLIST_IMAGE_FW_NAME);
break; break;
#endif #endif
#ifdef NVGPU_NETLIST_IMAGE_A #ifdef NVGPU_NETLIST_IMAGE_A
case NETLIST_SLOT_A: case NETLIST_SLOT_A:
(void) sprintf(name, NVGPU_NETLIST_IMAGE_A); (void) strcpy(name, NVGPU_NETLIST_IMAGE_A);
break; break;
#endif #endif
#ifdef NVGPU_NETLIST_IMAGE_B #ifdef NVGPU_NETLIST_IMAGE_B
case NETLIST_SLOT_B: case NETLIST_SLOT_B:
(void) sprintf(name, NVGPU_NETLIST_IMAGE_B); (void) strcpy(name, NVGPU_NETLIST_IMAGE_B);
break; break;
#endif #endif
#ifdef NVGPU_NETLIST_IMAGE_C #ifdef NVGPU_NETLIST_IMAGE_C
case NETLIST_SLOT_C: case NETLIST_SLOT_C:
(void) sprintf(name, NVGPU_NETLIST_IMAGE_C); (void) strcpy(name, NVGPU_NETLIST_IMAGE_C);
break; break;
#endif #endif
#ifdef NVGPU_NETLIST_IMAGE_D #ifdef NVGPU_NETLIST_IMAGE_D
case NETLIST_SLOT_D: case NETLIST_SLOT_D:
(void) sprintf(name, NVGPU_NETLIST_IMAGE_D); (void) strcpy(name, NVGPU_NETLIST_IMAGE_D);
break; break;
#endif #endif
default: default:

View File

@@ -30,8 +30,8 @@ int gv100_netlist_get_name(struct gk20a *g, int index, char *name)
switch (ver) { switch (ver) {
case NVGPU_GPUID_GV100: case NVGPU_GPUID_GV100:
(void) sprintf(name, "%s/%s", "gv100", (void) strcpy(name, "gv100/");
GV100_NETLIST_IMAGE_FW_NAME); (void) strcat(name, GV100_NETLIST_IMAGE_FW_NAME);
break; break;
default: default:
nvgpu_err(g, "no support for GPUID %x", ver); nvgpu_err(g, "no support for GPUID %x", ver);

View File

@@ -31,27 +31,27 @@ int gv11b_netlist_get_name(struct gk20a *g, int index, char *name)
switch (index) { switch (index) {
#ifdef GV11B_NETLIST_IMAGE_FW_NAME #ifdef GV11B_NETLIST_IMAGE_FW_NAME
case NETLIST_FINAL: case NETLIST_FINAL:
(void) sprintf(name, GV11B_NETLIST_IMAGE_FW_NAME); (void) strcpy(name, GV11B_NETLIST_IMAGE_FW_NAME);
break; break;
#endif #endif
#ifdef NVGPU_NETLIST_IMAGE_A #ifdef NVGPU_NETLIST_IMAGE_A
case NETLIST_SLOT_A: case NETLIST_SLOT_A:
(void) sprintf(name, NVGPU_NETLIST_IMAGE_A); (void) strcpy(name, NVGPU_NETLIST_IMAGE_A);
break; break;
#endif #endif
#ifdef NVGPU_NETLIST_IMAGE_B #ifdef NVGPU_NETLIST_IMAGE_B
case NETLIST_SLOT_B: case NETLIST_SLOT_B:
(void) sprintf(name, NVGPU_NETLIST_IMAGE_B); (void) strcpy(name, NVGPU_NETLIST_IMAGE_B);
break; break;
#endif #endif
#ifdef NVGPU_NETLIST_IMAGE_C #ifdef NVGPU_NETLIST_IMAGE_C
case NETLIST_SLOT_C: case NETLIST_SLOT_C:
(void) sprintf(name, NVGPU_NETLIST_IMAGE_C); (void) strcpy(name, NVGPU_NETLIST_IMAGE_C);
break; break;
#endif #endif
#ifdef NVGPU_NETLIST_IMAGE_D #ifdef NVGPU_NETLIST_IMAGE_D
case NETLIST_SLOT_D: case NETLIST_SLOT_D:
(void) sprintf(name, NVGPU_NETLIST_IMAGE_D); (void) strcpy(name, NVGPU_NETLIST_IMAGE_D);
break; break;
#endif #endif
default: default:

View File

@@ -31,8 +31,8 @@ int tu104_netlist_get_name(struct gk20a *g, int index, char *name)
switch (ver) { switch (ver) {
case NVGPU_GPUID_TU104: case NVGPU_GPUID_TU104:
(void) sprintf(name, "%s/%s", "tu104", (void) strcpy(name, "tu104/");
TU104_NETLIST_IMAGE_FW_NAME); (void) strcat(name, TU104_NETLIST_IMAGE_FW_NAME);
break; break;
default: default:
nvgpu_err(g, "no support for GPUID %x", ver); nvgpu_err(g, "no support for GPUID %x", ver);