We must have upper bound on what we are going to write into a fixed
size buffer.

Signed-off-by: Praveen Paneri <praveen.pan...@intel.com>
---
 intel/intel_decode.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/intel/intel_decode.c b/intel/intel_decode.c
index 7d5cbe5..b70d949 100644
--- a/intel/intel_decode.c
+++ b/intel/intel_decode.c
@@ -767,7 +767,7 @@ static void i915_get_instruction_src0(uint32_t *data, int 
i, char *srcname)
        char swizzle[100];
 
        i915_get_instruction_src_name((a0 >> 7) & 0x7, src_nr, srcname);
-       sprintf(swizzle, ".%s%s%s%s", swizzle_x, swizzle_y, swizzle_z,
+       snprintf(swizzle, sizeof(swizzle), ".%s%s%s%s", swizzle_x, swizzle_y, 
swizzle_z,
                swizzle_w);
        if (strcmp(swizzle, ".xyzw") != 0)
                strcat(srcname, swizzle);
@@ -785,7 +785,7 @@ static void i915_get_instruction_src1(uint32_t *data, int 
i, char *srcname)
        char swizzle[100];
 
        i915_get_instruction_src_name((a1 >> 13) & 0x7, src_nr, srcname);
-       sprintf(swizzle, ".%s%s%s%s", swizzle_x, swizzle_y, swizzle_z,
+       snprintf(swizzle, sizeof(swizzle), ".%s%s%s%s", swizzle_x, swizzle_y, 
swizzle_z,
                swizzle_w);
        if (strcmp(swizzle, ".xyzw") != 0)
                strcat(srcname, swizzle);
@@ -802,7 +802,7 @@ static void i915_get_instruction_src2(uint32_t *data, int 
i, char *srcname)
        char swizzle[100];
 
        i915_get_instruction_src_name((a2 >> 21) & 0x7, src_nr, srcname);
-       sprintf(swizzle, ".%s%s%s%s", swizzle_x, swizzle_y, swizzle_z,
+       snprintf(swizzle, sizeof(swizzle), ".%s%s%s%s", swizzle_x, swizzle_y, 
swizzle_z,
                swizzle_w);
        if (strcmp(swizzle, ".xyzw") != 0)
                strcat(srcname, swizzle);
@@ -931,7 +931,7 @@ i915_decode_dcl(struct drm_intel_decode *ctx, int i, char 
*instr_prefix)
 
        switch ((d0 >> 19) & 0x3) {
        case 1:
-               sprintf(dcl_mask, ".%s%s%s%s", dcl_x, dcl_y, dcl_z, dcl_w);
+               snprintf(dcl_mask, sizeof(dcl_mask), ".%s%s%s%s", dcl_x, dcl_y, 
dcl_z, dcl_w);
                if (strcmp(dcl_mask, ".") == 0)
                        fprintf(out, "bad (empty) dcl mask\n");
 
-- 
1.9.1

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to