Re: [Intel-gfx] [PATCH] drm/i915: silence coverity warning

2017-11-02 Thread Chris Wilson
Quoting Ville Syrjälä (2017-11-02 13:22:51) > On Thu, Nov 02, 2017 at 12:06:27PM +, Chris Wilson wrote: > > Quoting Lionel Landwerlin (2017-11-02 11:41:26) > > > Because dev_priv is 0-ed it's not currently an issue, but since we > > > have dev_priv->perf.oa.test_config.uuid size at uuid + 1, we

Re: [Intel-gfx] [PATCH] drm/i915: silence coverity warning

2017-11-02 Thread Ville Syrjälä
On Thu, Nov 02, 2017 at 12:06:27PM +, Chris Wilson wrote: > Quoting Lionel Landwerlin (2017-11-02 11:41:26) > > Because dev_priv is 0-ed it's not currently an issue, but since we > > have dev_priv->perf.oa.test_config.uuid size at uuid + 1, we could > > just copy the null character and silence

Re: [Intel-gfx] [PATCH] drm/i915: silence coverity warning

2017-11-02 Thread Chris Wilson
Quoting Lionel Landwerlin (2017-11-02 11:41:26) > Because dev_priv is 0-ed it's not currently an issue, but since we > have dev_priv->perf.oa.test_config.uuid size at uuid + 1, we could > just copy the null character and silence this warning. It doesn't copy the nul over exactly, to be complete s/

[Intel-gfx] [PATCH] drm/i915: silence coverity warning

2017-11-02 Thread Lionel Landwerlin
Because dev_priv is 0-ed it's not currently an issue, but since we have dev_priv->perf.oa.test_config.uuid size at uuid + 1, we could just copy the null character and silence this warning. Detected by CoverityScan, CID#369632 ("Buffer not null terminated") Signed-off-by: Lionel Landwerlin Cc: Ro