}
}
-char *sys_get_str(char *fname)
+int sys_get_str(char *fname, char* str)
{
- char buf[BUFF_MAX];
- char *r = NULL;
+ char buf[BUFF_MAX] = {0};
- if (sys_read_buf(fname, buf) == 0)
- r = strdup((char *)buf);
+ if (sys_read_buf(fname, buf) == 0) {
+ strncpy(str, buf, strlen(buf));
+ return 0;
+ }
- return r;
+ return -1;
}
int sys_set_int(char *fname, int val)
#define BUFF_MAX 255
#define MAX_NAME 255
+// TODO : checking return value
+#if 0
#define GENERATE_ACCESSORS_CHAR_RW(_suffix, _item) \
-char *OEM_sys_get_##_suffix() \
+char *OEM_sys_get_##_suffix(char* str) \
{ \
- return sys_get_str(_item); \
+ return sys_get_str(_item, str); \
} \
\
int OEM_sys_set_##_suffix(char *str) \
}
#define GENERATE_ACCESSORS_CHAR_R(_suffix, _item) \
-char *OEM_sys_get_##_suffix() \
+char *OEM_sys_get_##_suffix(char* str) \
{ \
- return sys_get_str(_item); \
+ return sys_get_str(_item, str); \
}
#define GENERATE_ACCESSORS_CHAR_W(_suffix, _item) \
{ \
return sys_set_str(_item, str); \
}
+#endif
/* TODO: Add APIs has (char *) params */
int OEM_sys_get_battery_health(int *value)
{
- char *buf;
- int i = 0;
+ char buf[BUFF_MAX] = {0};
+ int ret = 0;
+ int i = 0;
- buf = sys_get_str(BATTERY_HEALTH_PATH);
- if (NULL == buf) {
+ ret = sys_get_str(BATTERY_HEALTH_PATH, buf);
+ if (ret != 0) {
return -1;
}
int OEM_sys_get_uart_path(int *value)
{
- char *buf;
+ char buf[BUFF_MAX] = {0};
+ int ret = 0;
- buf = sys_get_str(UART_PATH);
- if (NULL == buf) {
+ ret = sys_get_str(UART_PATH, buf);
+ if (ret != 0) {
return -1;
}
int OEM_sys_get_usb_path(int *value)
{
- char *buf;
+ char buf[BUFF_MAX] = {0};
+ int ret = 0;
- buf = sys_get_str(USB_PATH);
- if (NULL == buf) {
+ ret = sys_get_str(USB_PATH, buf);
+ if (ret != 0) {
return -1;
}