Bug 561807 – inotify_sub.c :: dup_dirname() fails to remove trailing '/'

2008-11-28  Matthias Clasen  <mclasen@redhat.com>

        Bug 561807 – inotify_sub.c :: dup_dirname() fails to remove trailing
        '/'

        * inotify/inotify-sub.c (dup_dirname): Actually strip the trailing
        '/' away. Spotted by Dan Williams.

`

svn path=/trunk/; revision=7682

diff --git a/gio/ChangeLog b/gio/ChangeLog
index 97c3ea668fa9ec9ba3d396f284417e3c66bb4768..af3503e72fc09c813eb45f39f33ced3e44e632fe 100644 (file)
--- a/gio/ChangeLog
+++ b/gio/ChangeLog
@@ -1,3 +1,11 @@
+2008-11-28  Matthias Clasen  <mclasen@redhat.com>
+
+       Bug 561807 – inotify_sub.c :: dup_dirname() fails to remove trailing
+       '/'
+
+       * inotify/inotify-sub.c (dup_dirname): Actually strip the trailing
+       '/' away. Spotted by Dan Williams.
+
 2008-11-21  Cosimo Cecchi  <cosimoc@gnome.org>
 
        Bug 561375 – Leaks mountpoint description

diff --git a/gio/inotify/inotify-sub.c b/gio/inotify/inotify-sub.c
index 404dc208ef280c4c5b64f9aaa87d149412bc95a7..2b71c930da14c648a964bc16ffd99f9735a22d93 100644 (file)
--- a/gio/inotify/inotify-sub.c
+++ b/gio/inotify/inotify-sub.c
@@ -38,8 +38,8 @@ dup_dirname (const gchar *dirname)
   gchar *d_dirname = g_strdup (dirname);
   size_t len = strlen (d_dirname);
   
-  if (d_dirname[len] == '/')
-    d_dirname[len] = '\0';
+  if (d_dirname[len - 1] == '/')
+    d_dirname[len - 1] = '\0';
   
   return d_dirname;
 }