arpadboda commented on a change in pull request #557: MINIFICPP-850: Add
operating system to c2 response
URL: https://github.com/apache/nifi-minifi-cpp/pull/557#discussion_r285493254
##########
File path: libminifi/include/core/state/nodes/DeviceInformation.h
##########
@@ -468,6 +470,26 @@ class DeviceInfoNode : public DeviceInformation {
protected:
+ /**
+ * Have found various ways of identifying different operating system variants
+ * so these were either pulled from header files or online.
+ */
+ static inline std::string getOperatingSystem() {
+#ifdef _WIN32
+ return "Windows 32";
+#elif _WIN64
+ return "Windows 64";
+#elif __APPLE__ || __MACH__
Review comment:
Afaik __MACH__ is not Apple-exclusive, but that most probably won't hurt.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
