From 9c33ad84278353286bbb4118cd11ca3fcfec439a Mon Sep 17 00:00:00 2001
From: gcontini <1121667+gcontini@users.noreply.github.com>
Date: 周六, 31 10月 2020 23:36:25 +0800
Subject: [PATCH] codacy cleanup

---
 src/library/os/linux/execution_environment.cpp |   34 ++++++++++++++++------------------
 1 files changed, 16 insertions(+), 18 deletions(-)

diff --git a/src/library/os/linux/execution_environment.cpp b/src/library/os/linux/execution_environment.cpp
index 7df96e9..1ca0cd0 100644
--- a/src/library/os/linux/execution_environment.cpp
+++ b/src/library/os/linux/execution_environment.cpp
@@ -1,9 +1,10 @@
 /*
- * virtualization.cpp
+ * execution_environment.cpp
  *
  *  Created on: Dec 15, 2019
  *      Author: GC
  */
+#define __STDC_WANT_LIB_EXT1__1
 #include <paths.h>
 #include <sys/stat.h>
 #include <unistd.h>
@@ -18,7 +19,7 @@
 #include "../cpu_info.hpp"
 #include "../execution_environment.hpp"
 #include "../../base/file_utils.hpp"
-#include "../../base/StringUtils.h"
+#include "../../base/string_utils.h"
 
 namespace license {
 namespace os {
@@ -31,31 +32,28 @@
 	char path[MAX_PATH] = {0};
 	char proc_path[MAX_PATH], pidStr[64];
 	pid_t pid = getpid();
-	sprintf(pidStr, "%d", pid);
-	strcpy(proc_path, "/proc/");
-	strcat(proc_path, pidStr);
-	strcat(proc_path, "/cgroup");
+	snprintf(pidStr, sizeof(pidStr), "%d", pid);
+	strncpy(proc_path, "/proc/", sizeof(proc_path));
+	strncat(proc_path, pidStr, sizeof(proc_path));
+	strncpy(proc_path, "/cgroup", sizeof(proc_path));
 
 	FILE *fp;
 	char *line = NULL;
 	size_t len = 0;
 	ssize_t read;
-	CONTAINER_TYPE result = NONE;
+	CONTAINER_TYPE result = CONTAINER_TYPE::NONE;
 
 	fp = fopen(proc_path, "r");
 	if (fp == NULL) {
-		return NONE;
+		return CONTAINER_TYPE::NONE;
 	}
 
 	while ((read = getline(&line, &len, fp)) != -1 && result == 0) {
-		// line[len]=0;
-		// printf("Retrieved line of length %zu:\n", read);
-		// printf("%s", line);
 		if (strstr(line, "docker") != NULL) {
-			result = DOCKER;
+			result = CONTAINER_TYPE::DOCKER;
 		}
 		if (strstr(line, "lxc") != NULL) {
-			result = LXC;
+			result = CONTAINER_TYPE::LXC;
 		}
 	}
 
@@ -67,15 +65,15 @@
 // 0=NO 1=Docker/2=Lxc
 static CONTAINER_TYPE checkSystemdContainer() {
 	ifstream systemd_container("/var/run/systemd/container");
-	CONTAINER_TYPE result = NONE;
+	CONTAINER_TYPE result = CONTAINER_TYPE::NONE;
 	if (systemd_container.good()) {
-		result = DOCKER;
+		result = CONTAINER_TYPE::DOCKER;
 		for (string line; getline(systemd_container, line);) {
 			if (line.find("docker") != string::npos) {
-				result = DOCKER;
+				result = CONTAINER_TYPE::DOCKER;
 				break;
 			} else if (line.find("lxc") != string::npos) {
-				result = LXC;
+				result = CONTAINER_TYPE::LXC;
 				break;
 			}
 		}
@@ -85,7 +83,7 @@
 
 static CONTAINER_TYPE get_container_type() {
 	CONTAINER_TYPE result = checkContainerProc();
-	if (result == NONE) {
+	if (result == CONTAINER_TYPE::NONE) {
 		result = checkSystemdContainer();
 	}
 	return result;

--
Gitblit v1.9.1