aurum: fix using new/delete instead of variable-sized array. 88/285488/3 accepted/tizen/unified/20230105.154713
authorHosang Kim <hosang12.kim@samsung.com>
Tue, 13 Dec 2022 11:59:08 +0000 (20:59 +0900)
committerkim hosang <hosang12.kim@samsung.com>
Tue, 13 Dec 2022 11:59:52 +0000 (11:59 +0000)
This reverts commit 0f2e8dac34da4f1e5c4d6d0a3537226dd9e99f10.

Change-Id: I39024c57526ce3d060d4744a18a013c9903588d3

org.tizen.aurum-bootstrap/src/Commands/TakeScreenshotCommand.cc

index 2a6245790489187ddb5d0439b360d82ed704c47b..5825eef022255cd993e8ccfc87eea0248dc60ebc 100644 (file)
@@ -49,8 +49,9 @@ TakeScreenshotCommand::TakeScreenshotCommand(
 
     std::ifstream ifs(path, std::ifstream::binary);
     ::aurum::RspTakeScreenshot rsp;
-    int size = mDevice->getScreenSize().width * mDevice->getScreenSize().height;
-    char *buf = (char *) calloc(size, sizeof(char));
+    const Size2D<int> screenSize = mDevice->getScreenSize();
+    int size = screenSize.width * screenSize.height;
+    char *buf = new char[size];
 
     while (!ifs.eof()) {
         ifs.read(buf, size);
@@ -58,6 +59,7 @@ TakeScreenshotCommand::TakeScreenshotCommand(
         mWriter->Write(rsp);
     }
     ifs.close();
+    delete[] buf;
 
     return grpc::Status::OK;
 }