projects
/
model-checker.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
Fix snapshot code
[model-checker.git]
/
snapshot-interface.cc
diff --git
a/snapshot-interface.cc
b/snapshot-interface.cc
index cd1eceea744b2262f2899542aa59dd0ece45db88..5242d168fb4072552cda80f5069b85c3a29fd95b 100644
(file)
--- a/
snapshot-interface.cc
+++ b/
snapshot-interface.cc
@@
-2,17
+2,16
@@
#include <unistd.h>
#include <cstring>
#include <inttypes.h>
#include <unistd.h>
#include <cstring>
#include <inttypes.h>
-#include <vector>
#include "snapshot-interface.h"
#include "snapshot.h"
#include "common.h"
#include "mymemory.h"
#include "snapshot-interface.h"
#include "snapshot.h"
#include "common.h"
#include "mymemory.h"
+#include "stl-model.h"
/* MYBINARYNAME only works because our pathname usually includes 'model' (e.g.,
* /.../model-checker/test/userprog.o) */
#define MYBINARYNAME "model"
/* MYBINARYNAME only works because our pathname usually includes 'model' (e.g.,
* /.../model-checker/test/userprog.o) */
#define MYBINARYNAME "model"
-#define MYLIBRARYNAME "libmodel.so"
#define MAPFILE "/proc/self/maps"
struct snapshot_entry {
#define MAPFILE "/proc/self/maps"
struct snapshot_entry {
@@
-29,7
+28,7
@@
class SnapshotStack {
MEMALLOC
private:
MEMALLOC
private:
-
std::vector<struct snapshot_entry, ModelAlloc<struct snapshot_entry>
> stack;
+
ModelVector<struct snapshot_entry
> stack;
};
static SnapshotStack *snap_stack;
};
static SnapshotStack *snap_stack;
@@
-65,16
+64,18
@@
static void SnapshotGlobalSegments()
char smstr[23];
char r, w, x;
char mr, mw, mx;
char smstr[23];
char r, w, x;
char mr, mw, mx;
- int size;
void *begin, *end;
//Skip out at the end of the section
if (buf[0] == '\n')
break;
void *begin, *end;
//Skip out at the end of the section
if (buf[0] == '\n')
break;
- sscanf(buf, "%22s %p-%p
[%5dK] %c%c%c/%c%c%c SM=%3s %200s\n", type, &begin, &end, &size, &r, &w, &x, &mr, &mw, &mx, smstr, regionname
);
+ sscanf(buf, "%22s %p-%p
", type, &begin, &end
);
- if (w == 'w' && (strstr(regionname, MYBINARYNAME) || strstr(regionname, MYLIBRARYNAME))) {
+ char * secondpart = strstr(buf, "]");
+
+ sscanf(&secondpart[2], "%c%c%c/%c%c%c SM=%3s %200s\n", &r, &w, &x, &mr, &mw, &mx, smstr, regionname);
+ if (w == 'w' && strstr(regionname, MYBINARYNAME)) {
size_t len = ((uintptr_t)end - (uintptr_t)begin) / PAGESIZE;
if (len != 0)
snapshot_add_memory_region(begin, len);
size_t len = ((uintptr_t)end - (uintptr_t)begin) / PAGESIZE;
if (len != 0)
snapshot_add_memory_region(begin, len);
@@
-86,10
+87,16
@@
static void SnapshotGlobalSegments()
static void get_binary_name(char *buf, size_t len)
{
static void get_binary_name(char *buf, size_t len)
{
- if (readlink("/proc/self/exe", buf, len) == -1) {
+ ssize_t size = readlink("/proc/self/exe", buf, len);
+ if (size < 0) {
perror("readlink");
exit(EXIT_FAILURE);
}
perror("readlink");
exit(EXIT_FAILURE);
}
+
+ /* Terminate string */
+ if ((size_t)size > len)
+ size = len;
+ buf[size] = '\0';
}
/** The SnapshotGlobalSegments function computes the memory regions
}
/** The SnapshotGlobalSegments function computes the memory regions
@@
-114,7
+121,7
@@
static void SnapshotGlobalSegments()
void *begin, *end;
sscanf(buf, "%p-%p %c%c%c%c %*x %*x:%*x %*u %200s\n", &begin, &end, &r, &w, &x, &p, regionname);
void *begin, *end;
sscanf(buf, "%p-%p %c%c%c%c %*x %*x:%*x %*u %200s\n", &begin, &end, &r, &w, &x, &p, regionname);
- if (w == 'w' &&
(strstr(regionname, binary_name) || strstr(regionname, MYLIBRARYNAME)
)) {
+ if (w == 'w' &&
strstr(regionname, binary_name
)) {
size_t len = ((uintptr_t)end - (uintptr_t)begin) / PAGESIZE;
if (len != 0)
snapshot_add_memory_region(begin, len);
size_t len = ((uintptr_t)end - (uintptr_t)begin) / PAGESIZE;
if (len != 0)
snapshot_add_memory_region(begin, len);
@@
-148,7
+155,7
@@
int SnapshotStack::backTrackBeforeStep(int seqindex)
/** This method takes a snapshot at the given sequence number. */
void SnapshotStack::snapshotStep(int seqindex)
{
/** This method takes a snapshot at the given sequence number. */
void SnapshotStack::snapshotStep(int seqindex)
{
- stack.push_back(snapshot_entry(
seqindex, take_snapshot()
));
+ stack.push_back(snapshot_entry(
take_snapshot(), seqindex
));
}
void snapshot_stack_init()
}
void snapshot_stack_init()