ini: not using getenv

[platform/core/uifw/libtdm.git] / src / tdm_helper.c

diff --git a/src/tdm_helper.c b/src/tdm_helper.c
index 74d8bb5..e3d5f3f 100644 (file)
--- a/src/tdm_helper.c
+++ b/src/tdm_helper.c
@@ -38,18 +38,9 @@
 #endif
 
 #include <png.h>
-#include <string.h>
-#include <tbm_surface.h>
-#include <tbm_surface_internal.h>
-#include <tbm_drm_helper.h>
-#include <string.h>
-#include <time.h>
 #include <pixman.h>
-#include <inttypes.h>
 
-#include "tdm.h"
 #include "tdm_private.h"
-#include "tdm_helper.h"
 
 #define PNG_DEPTH 8
 
@@ -70,26 +61,20 @@ tdm_helper_get_time(void)
 }
 
 static int
-_tdm_helper_check_file_is_valid(const char* path, int del_link)
+_tdm_helper_check_file_is_symbolic_link(const char* path)
 {
-       char *real_path;
+       struct stat sb;
 
        if (!path)
                return 0;
 
-       real_path = realpath(path, NULL);
-       if (real_path && strncmp(path, real_path, strlen(path))) {
-               if (del_link)
-                       unlink(path);
-               free(real_path);
-
+       if (stat(path, &sb) != 0)
                return 0;
-       }
 
-       if (real_path)
-               free(real_path);
+       if (S_ISLNK(sb.st_mode))
+               return 1;
 
-       return 1;
+       return 0;
 }
 
 static void
@@ -99,8 +84,10 @@ _tdm_helper_dump_raw(const char *file, void *data1, int size1, void *data2,
        unsigned int *blocks;
        FILE *fp;
 
-       if (!_tdm_helper_check_file_is_valid(file, 1))
-               TDM_WRN("'%s' may be symbolic link\n", file);
+       if (_tdm_helper_check_file_is_symbolic_link(file)) {
+               TDM_ERR("'%s' may be symbolic link\n", file);
+               return;
+       }
 
        fp = fopen(file, "w+");
        TDM_RETURN_IF_FAIL(fp != NULL);
@@ -127,8 +114,10 @@ _tdm_helper_dump_png(const char *file, const void *data, int width,
 {
        FILE *fp;
 
-       if (!_tdm_helper_check_file_is_valid(file, 1))
-               TDM_WRN("'%s' may be symbolic link\n", file);
+       if (_tdm_helper_check_file_is_symbolic_link(file)) {
+               TDM_ERR("'%s' may be symbolic link\n", file);
+               return;
+       }
 
        fp = fopen(file, "wb");
        TDM_RETURN_IF_FAIL(fp != NULL);
@@ -246,30 +235,26 @@ failed_make:
        return NULL;
 }
 
-INTERN void
+EXTERN void
 tdm_helper_dump_buffer_str(tbm_surface_h buffer, char *dir, char *str)
 {
        tbm_surface_info_s info;
-       const char *ext;
        char file[TDM_PATH_LEN];
        int ret, bw, bh;
 
        TDM_RETURN_IF_FAIL(buffer != NULL);
-       TDM_RETURN_IF_FAIL(dir != NULL);
        TDM_RETURN_IF_FAIL(str != NULL);
 
+       if (!dir)
+               dir = ".";
+
        ret = tbm_surface_get_info(buffer, &info);
        TDM_RETURN_IF_FAIL(ret == TBM_SURFACE_ERROR_NONE);
 
-       if (IS_RGB(info.format))
-               ext = file_exts[0];
-       else
-               ext = file_exts[1];
-
        tdm_helper_get_buffer_full_size(buffer, &bw, &bh);
 
-       snprintf(file, TDM_PATH_LEN, "%s/%c%c%c%c_%dx%d_%dx%d_%s.%s",
-                        dir, FOURCC_STR(info.format), bw, bh, info.width, info.height, str, ext);
+       snprintf(file, TDM_PATH_LEN, "%s/%c%c%c%c_%dx%d_%dx%d_%s",
+                        dir, FOURCC_STR(info.format), bw, bh, info.width, info.height, str);
 
        tdm_helper_dump_buffer(buffer, file);
 }
@@ -278,11 +263,13 @@ tdm_helper_dump_buffer_str(tbm_surface_h buffer, char *dir, char *str)
 EXTERN void
 tdm_helper_dump_buffer(tbm_surface_h buffer, const char *file)
 {
+       char temp[TDM_PATH_LEN] = {0,};
        tbm_surface_info_s info;
        int len, ret;
        const char *ext;
        int bo_cnt;
        int bw, bh;
+       char *dot, *p = temp;
 
        TDM_RETURN_IF_FAIL(buffer != NULL);
        TDM_RETURN_IF_FAIL(file != NULL);
@@ -290,16 +277,20 @@ tdm_helper_dump_buffer(tbm_surface_h buffer, const char *file)
        ret = tbm_surface_map(buffer, TBM_OPTION_READ, &info);
        TDM_RETURN_IF_FAIL(ret == TBM_SURFACE_ERROR_NONE);
 
-       len = strnlen(file, 1024);
        if (IS_RGB(info.format))
                ext = file_exts[0];
        else
                ext = file_exts[1];
 
-       if (strncmp(file + (len - 3), ext, 3)) {
-               TDM_ERR("can't dump to '%s' file", file + (len - 3));
-               tbm_surface_unmap(buffer);
-               return;
+       dot = strrchr(file, '.');
+       if (!dot || strlen(dot + 1) != 3 || strncmp(dot + 1, ext, 3)) {
+               len = strnlen(file, TDM_PATH_LEN - 5);
+               strncat(p, file, len);
+               p += len;
+               *(p++) = '.';
+               strncat(p, ext, 3);
+               p += 3;
+               *p = '\0';
        }
 
        tdm_helper_get_buffer_full_size(buffer, &bw, &bh);
@@ -314,11 +305,11 @@ tdm_helper_dump_buffer(tbm_surface_h buffer, const char *file)
        switch (info.format) {
        case TBM_FORMAT_ARGB8888:
        case TBM_FORMAT_XRGB8888:
-               _tdm_helper_dump_png(file, info.planes[0].ptr, bw, bh);
+               _tdm_helper_dump_png(temp, info.planes[0].ptr, bw, bh);
                break;
        case TBM_FORMAT_YVU420:
        case TBM_FORMAT_YUV420:
-               _tdm_helper_dump_raw(file,
+               _tdm_helper_dump_raw((const char*)temp,
                                                         info.planes[0].ptr,
                                                         info.planes[0].size,
                                                         info.planes[1].ptr,
@@ -328,7 +319,7 @@ tdm_helper_dump_buffer(tbm_surface_h buffer, const char *file)
                break;
        case TBM_FORMAT_NV12:
        case TBM_FORMAT_NV21:
-               _tdm_helper_dump_raw(file,
+               _tdm_helper_dump_raw((const char*)temp,
                                                         info.planes[0].ptr,
                                                         info.planes[0].size,
                                                         info.planes[1].ptr,
@@ -337,7 +328,7 @@ tdm_helper_dump_buffer(tbm_surface_h buffer, const char *file)
                break;
        case TBM_FORMAT_YUYV:
        case TBM_FORMAT_UYVY:
-               _tdm_helper_dump_raw(file,
+               _tdm_helper_dump_raw((const char*)temp,
                                                         info.planes[0].ptr,
                                                         info.planes[0].size, NULL, 0,
                                                         NULL, 0);
@@ -350,7 +341,7 @@ tdm_helper_dump_buffer(tbm_surface_h buffer, const char *file)
 
        tbm_surface_unmap(buffer);
 
-       TDM_INFO("dump %s", file);
+       TDM_INFO("dump %s", temp);
 }
 
 EXTERN void
@@ -732,7 +723,6 @@ tdm_helper_capture_output(tdm_output *output, tbm_surface_h dst_buffer,
        TDM_RETURN_VAL_IF_FAIL(w >= 0, TDM_ERROR_INVALID_PARAMETER);
        TDM_RETURN_VAL_IF_FAIL(h >= 0, TDM_ERROR_INVALID_PARAMETER);
        TDM_RETURN_VAL_IF_FAIL(func != NULL, TDM_ERROR_INVALID_PARAMETER);
-       TDM_RETURN_VAL_IF_FAIL(data != NULL, TDM_ERROR_INVALID_PARAMETER);
 
        err = tdm_output_get_layer_count(output, &count);
        if (err != TDM_ERROR_NONE) {