From ba139824fa06a97f2a3b1cc4c6085d10a83ec2b9 Mon Sep 17 00:00:00 2001
From: Jeremie Koenig <jk@jk.fr.eu.org>
Date: Tue, 17 Aug 2010 17:27:54 +0000
Subject: More cleanup possibilities

* procfs.c, procfs.h: Extend the signature of the
  cleanup_contents callback in the procfs_node_ops structure to include
  the hook and contents_len.  (cleanup_contents_with_free,
  cleanup_contents_with_vm_deallocate): New functions, can be used as
  a cleanup_contents callback for simple cases.
* procfs_dir.c, procfs_dir.h (procfs_dir_make_node):
  Update, add a cleanup callback, make sure the cleanup callback is
  invoked if there is an error.
* proclist.c (proclist_make_node), main.c (main): Update to match
  the new interfaces.
---
 procfs.c | 15 ++++++++++++++-
 1 file changed, 14 insertions(+), 1 deletion(-)

(limited to 'procfs.c')

diff --git a/procfs.c b/procfs.c
index 0c57686b..755e0519 100644
--- a/procfs.c
+++ b/procfs.c
@@ -1,6 +1,7 @@
 #include <stdlib.h>
 #include <string.h>
 #include <fcntl.h>
+#include <mach.h>
 #include <hurd/netfs.h>
 #include <hurd/fshelp.h>
 #include "procfs.h"
@@ -15,6 +16,18 @@ struct netnode
   size_t contents_len;
 };
 
+void
+procfs_cleanup_contents_with_free (void *hook, void *cont, size_t len)
+{
+  free (cont);
+}
+
+void
+procfs_cleanup_contents_with_vm_deallocate (void *hook, void *cont, size_t len)
+{
+  vm_deallocate (mach_task_self (), (vm_address_t) cont, (vm_size_t) len);
+}
+
 struct node *procfs_make_node (const struct procfs_node_ops *ops, void *hook)
 {
   struct netnode *nn;
@@ -85,7 +98,7 @@ error_t procfs_lookup (struct node *np, const char *name, struct node **npp)
 void procfs_cleanup (struct node *np)
 {
   if (np->nn->contents && np->nn->ops->cleanup_contents)
-    np->nn->ops->cleanup_contents (np->nn->contents);
+    np->nn->ops->cleanup_contents (np->nn->hook, np->nn->contents, np->nn->contents_len);
 
   if (np->nn->ops->cleanup)
     np->nn->ops->cleanup (np->nn->hook);
-- 
cgit v1.2.3