organic groups integration leaks private groups

the default implementation doesnt check if a group is set to private before including it on the list

there should be some way to configure this per role but OG lacks it and i might take this to them if i get a chance to work on it, so the below might need some work on OG before it will be useful, i think it should be included anyway because it obeys OG's logic
im actually using the below + a check for role but that hack doesnt belong to this patch :-)

--- author_pane/modules/og.author-pane.inc 2009-09-02 14:04:08.000000000 -0700
+++ author_pane/modules/og.author-pane.inc      2009-09-02 14:08:22.000000000 -0700
@@ -15,7 +15,10 @@ function og_preprocess_author_pane(&$var
   if (isset($account->og_groups) && !empty($account->og_groups)) {
     $groups = array();
     foreach ($account->og_groups as $og_id => $og) {
-      $groups[] = l($og['title'], 'node/' . $og['nid']);
+      $og_node = node_load($og['nid']) ;
+      if ( ! $og_node['og_private'] ) {
+        $groups[] = l($og['title'], 'node/' . $og['nid']);
+      }
     }

     $variables['og_groups'] = implode(', ', $groups);