diff options
| author | hding3 <haitao.ding@intel.com> | 2013-04-23 18:11:03 +0800 |
|---|---|---|
| committer | buildbot <buildbot@intel.com> | 2013-04-25 08:52:15 -0700 |
| commit | c31b44d6c9589c1dcc48d86ad72aec89760433bc (patch) | |
| tree | 034839497e22b9d59e31f57bac84fa7d1a3b756c /va | |
| parent | 85d48b6ba08430292e4c47df2898fab3042c7ba3 (diff) | |
| download | libva-c31b44d6c9589c1dcc48d86ad72aec89760433bc.tar.gz | |
libva: fix one klocwork issue in function va_DisplayContextGetDriverName.
BZ: 102729
remove unnecessay setting for *driver_name.
Change-Id: I1fe19edca46cc1796f363e6ae72401d35e8812a2
Signed-off-by: hding3 <haitao.ding@intel.com>
Signed-off-by: Fei Jiang <fei.jiang@intel.com>
Reviewed-on: http://android.intel.com:8080/104229
Reviewed-by: Shi, PingX <pingx.shi@intel.com>
Reviewed-by: cactus <cactus@intel.com>
Tested-by: Shi, PingX <pingx.shi@intel.com>
Reviewed-by: buildbot <buildbot@intel.com>
Tested-by: buildbot <buildbot@intel.com>
Diffstat (limited to 'va')
| -rw-r--r-- | va/android/va_android.cpp | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/va/android/va_android.cpp b/va/android/va_android.cpp index 97c884c..3e220a6 100644 --- a/va/android/va_android.cpp +++ b/va/android/va_android.cpp @@ -115,11 +115,13 @@ static VAStatus va_DisplayContextGetDriverName ( return VA_STATUS_ERROR_UNKNOWN; } drm_state->auth_type = VA_DRM_AUTH_CUSTOM; - *driver_name = NULL; + + if (driver_name == NULL) + return VA_STATUS_ERROR_UNKNOWN; if (strncmp((char *)ctx->native_dpy, "libva_driver_name=", 18) == 0) { - *driver_name = strdup((char *)ctx->native_dpy + 18); - if (!driver_name) + *driver_name = strdup((char *)ctx->native_dpy + 18); + if (*driver_name == NULL) return VA_STATUS_ERROR_ALLOCATION_FAILED; else return VA_STATUS_SUCCESS; |