diff options
| author | Xiang, Haihao <haihao.xiang@intel.com> | 2013-08-12 14:52:14 +0800 |
|---|---|---|
| committer | Gwenole Beauchesne <gwenole.beauchesne@intel.com> | 2013-08-27 11:16:45 +0200 |
| commit | 43c3a2f79d63e638e0765af66ad650d6ba45e163 (patch) | |
| tree | 8376fb877540f503be36607ceda7667bee37d116 | |
| parent | 7d41346c9f1ce48c5e7282d28e98ab279ee9b247 (diff) | |
| download | libva-43c3a2f79d63e638e0765af66ad650d6ba45e163.tar.gz | |
vatrace: check the pointer against NULL before using it
Signed-off-by: Xiang, Haihao <haihao.xiang@intel.com>
| -rw-r--r-- | va/va_trace.c | 17 |
1 files changed, 9 insertions, 8 deletions
diff --git a/va/va_trace.c b/va/va_trace.c index cfc7c5c..f288d49 100644 --- a/va/va_trace.c +++ b/va/va_trace.c @@ -359,6 +359,9 @@ void va_TraceSurface(VADisplay dpy) unsigned char check_sum = 0; DPY2TRACECTX(dpy); + if (!trace_ctx->trace_fp_surface) + return; + va_TraceMsg(trace_ctx, "==========dump surface data in file %s\n", trace_ctx->trace_surface_fn); if ((file_size(trace_ctx->trace_fp_surface) >= trace_logsize)) { @@ -405,20 +408,18 @@ void va_TraceSurface(VADisplay dpy) tmp = Y_data + luma_stride * trace_ctx->trace_surface_yoff; for (i=0; i<trace_ctx->trace_surface_height; i++) { - if (trace_ctx->trace_fp_surface) - fwrite(tmp + trace_ctx->trace_surface_xoff, - trace_ctx->trace_surface_width, - 1, trace_ctx->trace_fp_surface); + fwrite(tmp + trace_ctx->trace_surface_xoff, + trace_ctx->trace_surface_width, + 1, trace_ctx->trace_fp_surface); tmp += luma_stride; } tmp = UV_data + chroma_u_stride * trace_ctx->trace_surface_yoff; if (fourcc == VA_FOURCC_NV12) { for (i=0; i<trace_ctx->trace_surface_height/2; i++) { - if (trace_ctx->trace_fp_surface) - fwrite(tmp + trace_ctx->trace_surface_xoff, - trace_ctx->trace_surface_width, - 1, trace_ctx->trace_fp_surface); + fwrite(tmp + trace_ctx->trace_surface_xoff, + trace_ctx->trace_surface_width, + 1, trace_ctx->trace_fp_surface); tmp += chroma_u_stride; } |