meson_display: Need to close fd [1/1]
PD#SWPL-199491
Problem:
Need to close fd
Solution:
Need to close fd
Verify:
s7d
Change-Id: I5d4bbf0eae9ba723f3c838a8cdde13987e570569
Signed-off-by: chen.wang1 <chen.wang1@amlogic.com>
diff --git a/display_framework/src/extension/display_settings/displayattribute/display_attribute.c b/display_framework/src/extension/display_settings/displayattribute/display_attribute.c
index bd19052..d72d7ca 100644
--- a/display_framework/src/extension/display_settings/displayattribute/display_attribute.c
+++ b/display_framework/src/extension/display_settings/displayattribute/display_attribute.c
@@ -79,11 +79,11 @@
}
void getDisplayEDIDData(DISPLAY_CONNECTOR_TYPE connType, int * data_Len, char **data ) {
- int fd = display_meson_get_open();
if (data_Len == NULL || data == NULL) {
ERROR("%s %d invalid parameter return",__FUNCTION__,__LINE__);
return;
}
+ int fd = display_meson_get_open();
meson_drm_getEDIDData(fd, connType, data_Len, data);
DEBUG_EDID("\n");
DEBUG("%s %d get data_Len: %d",__FUNCTION__,__LINE__, (*data_Len));
@@ -509,6 +509,7 @@
displayAspectRatio = DISPLAY_ASPECT_RATIO_RESERVED;
break;
}
+ meson_close_drm(fd);
DEBUG("%s %d get aspect ratio %s",__FUNCTION__,__LINE__,str);
return displayAspectRatio;
}
@@ -528,7 +529,6 @@
bool hdrStatus = false;
char color[5] = {'\0'};
int supportedcheck = -1;
- int fd = display_meson_get_open();
ENUM_DISPLAY_HDR_MODE displayHdrMode = getDisplayHdrStatus(connType);
if (displayHdrMode == MESON_DISPLAY_HDR10PLUS || displayHdrMode == MESON_DISPLAY_HDR10_ST2084 ||
displayHdrMode == MESON_DISPLAY_HDR10_TRADITIONAL ||displayHdrMode == MESON_DISPLAY_HDR_HLG) {