fix test for osx #123, remove warnings and simplify impl

Author: planrich

src/_vmprof.c

@@ -442,7 +442,8 @@ resolve_addr(PyObject *module, PyObject *args) {
 static PyObject * vmp_get_profile_path(PyObject *module, PyObject *noargs) {
     PyObject * o;
     if (is_enabled) {
-        char * buffer[4096];
+        char buffer[4096];
+        buffer[0] = 0;
         ssize_t buffer_len = vmp_fd_to_path(vmp_profile_fileno(), buffer, 4096);
         if (buffer_len == -1) {
             PyErr_Format(PyExc_NotImplementedError, "not implemented platform %s", vmp_machine_os_name());

src/machine.c

@@ -3,6 +3,11 @@
 #include "vmprof.h"
 #include <stdio.h>
 
+#ifdef VMPROF_UNIX
+#include <unistd.h>
+#include <fcntl.h>
+#endif
+
 int vmp_machine_bits(void)
 {
     return sizeof(void*)*8;
@@ -34,12 +39,8 @@ long vmp_fd_to_path(int fd, char * buffer, long buffer_len)
     (void)snprintf(proffs, 24, "/proc/self/fd/%d", fd);
     return readlink(proffs, buffer, buffer_len);
 #elif defined(VMPROF_UNIX)
-    char buf[MAXPATHLEN];
-    int size = fcntl(fd, F_GETPATH, buf);
-    if (size >= 0) {
-        (void)strncpy(buffer, buf, buffer_len);
-    }
-    return size;
+    fcntl(fd, F_GETPATH, buffer);
+    return strlen(buffer);
 #endif
     return -1;
 }

src/symboltable.h

@@ -17,5 +17,7 @@
  */
 void dump_all_known_symbols(int fd);
 
+void dump_native_symbols(int fd);
+
 int vmp_resolve_addr(void * addr, char * name, int name_len, int * lineno,
                       char * srcfile, int srcfile_len);

vmprof/test/test_run.py

@@ -433,7 +433,10 @@ def test_get_profile_path(self):
         assert vmprof.get_profile_path() == None
         tmpfile = tempfile.NamedTemporaryFile(delete=False)
         vmprof.enable(tmpfile.fileno())
-        assert vmprof.get_profile_path() == tmpfile.name
+        if not vmprof.get_profile_path() == tmpfile.name:
+            with open(vmprof.get_profile_path(), 'rb') as fd1:
+                with open(tmpfile.name, "rb") as fd2:
+                    assert fd1.read() == fd2.read()
         vmprof.disable()
         assert vmprof.get_profile_path() == None