[PATCH openpower-host-ipmi-oem 7/8] Fixed string pointer usage
OpenBMC Patches
patches at stwcx.xyz
Thu Oct 22 13:52:12 AEDT 2015
From: Chris Austen <austenc at us.ibm.com>
---
oemhandler.C | 7 ++-----
testit.C | 8 +++-----
2 files changed, 5 insertions(+), 10 deletions(-)
diff --git a/oemhandler.C b/oemhandler.C
index 427831c..0cc7099 100644
--- a/oemhandler.C
+++ b/oemhandler.C
@@ -11,9 +11,6 @@ uint16_t g_record_id = 0x0100;
#define LSMSSWAP(x,y) ((y)<<8|(x))
-const char *fio_wb = "wb";
-const char *fio_rb = "rb+";
-
///////////////////////////////////////////////////////////////////////////////
// For the First partial add eSEL the SEL Record ID and offset
@@ -61,9 +58,9 @@ ipmi_ret_t ipmi_ibm_oem_partial_esel(ipmi_netfn_t netfn, ipmi_cmd_t cmd,
// to indicate this is a new record rather then an ofset in
// my next commit TODO
if (offset == 0) {
- pio = fio_wb;
+ pio = "wb";
} else {
- pio = fio_rb;
+ pio = "rb+";
}
if ((fp = fopen(string, pio)) != NULL) {
diff --git a/testit.C b/testit.C
index 25e069c..2b0005e 100644
--- a/testit.C
+++ b/testit.C
@@ -1,12 +1,10 @@
-#include <iostream>
#include <cstdlib>
-#include "ipmid-api.h"
-#include "oemhandler.h"
#include <cstring>
-
-#include <iostream>
#include <fstream>
+#include <iostream>
#include <vector>
+#include "ipmid-api.h"
+#include "oemhandler.h"
using namespace std;
--
2.6.0
More information about the openbmc
mailing list