[ltt-dev] [UST PATCH] Get Online targets checker

[Matthew Khouzam]

Added a check in get online pids to only send the pids that are 
currently online.


Signed-off-by: Matthew Khouzam <matthew.khouzam at ericsson.com>
---
  libustctl/libustctl.c |    9 ++++++++-
  1 files changed, 8 insertions(+), 1 deletions(-)

diff --git a/libustctl/libustctl.c b/libustctl/libustctl.c
index d57e645..3b5ae84 100644
--- a/libustctl/libustctl.c
+++ b/libustctl/libustctl.c
@@ -92,8 +92,11 @@ pid_t *ustctl_get_online_pids(void)
  {
      struct dirent *dirent;
      DIR *dir;
+    DIR *proc_dir;
+    char proc_dir_path[80]; /* proc+ / + int always smaller than 80*/
      unsigned int ret_size = 1 * sizeof(pid_t), i = 0;

+    printf( "%s\n" , SOCK_DIR) ;
      dir = opendir(SOCK_DIR);
      if (!dir) {
          return NULL;
@@ -117,7 +120,11 @@ pid_t *ustctl_get_online_pids(void)
               * We need to figure out an intelligent way of solving
               * this, maybe connect-disconnect.
               */
-            if (1) {
+             snprintf( proc_dir_path, 80, "/proc/%u/" , ret[i]) ;
+             proc_dir = opendir(proc_dir_path);
+             /* maybe we should remove the empty dir in this pass too*/
+             if( proc_dir != NULL )
+            {
                  ret_size += sizeof(pid_t);
                  ret = (pid_t *) realloc(ret, ret_size);
                  ++i;
-- 
1.7.0.4

-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Added-a-check-in-get-online-pids-to-only-send-the-pi.patch
Type: text/x-patch
Size: 1361 bytes
Desc: not available
URL: <http://lists.casi.polymtl.ca/pipermail/lttng-dev/attachments/20110325/1043184f/attachment-0003.bin>