cleanup and cosmetics
authorChristian Thaeter <ct@pipapo.org>
Wed, 2 Jan 2008 18:04:54 +0000 (19:04 +0100)
committerChristian Thaeter <ct@pipapo.org>
Wed, 2 Jan 2008 18:04:54 +0000 (19:04 +0100)
add comments about to be done input validation and remove the libgit
prototypes in favor of including git headers
(TODO: add checks to configure)

src/actions.c
src/ctgit.h
src/query.c

index 7be07dc..953a94d 100644 (file)
@@ -68,6 +68,8 @@ ctgit_main_content_action (struct ctgit_query* query)
                                           html(
                                                ctgit_log_link (query, n, n->last_head, html("Log")),
                                                " ",
+                                               //ctgit_diff_link (query, n, n->last_head, html("Diff")),
+                                               //" ",
                                                ctgit_object_link (query, n, n->last_tree, html("Tree"))
                                                )
                                           )
index 3849f2a..1020a06 100644 (file)
@@ -76,11 +76,4 @@ ctgit_git_open (const char* fmt, ...);
 void
 ctgit_git_close (FILE* handle);
 
-/*
-  libgit api
-*/
-typedef int (*config_fn_t)(const char *, const char *);
-extern const char* setup_git_directory (void);
-extern int git_config(config_fn_t fn);
-
 #endif
index 75e8734..cd67b1e 100644 (file)
@@ -83,6 +83,8 @@ ctgit_repo_param (const Cgi self, const char* v, size_t v_sz, void* u_dat)
   (void) self;
   struct ctgit_query* q = (struct ctgit_query*) u_dat;
 
+  /* TODO validate that v is a probably legal repo name (alnum()) */
+
   free (q->repo);
   q->repo = cwa_strndup (v, v_sz);
 }
@@ -94,6 +96,8 @@ ctgit_action_param (const Cgi self, const char* v, size_t v_sz, void* u_dat)
   (void) self;
   struct ctgit_query* q = (struct ctgit_query*) u_dat;
 
+  /* TODO validate that v is a probably legal action name alpha() */
+
   free (q->action);
   q->action = cwa_strndup (v, v_sz);
 }
@@ -108,6 +112,8 @@ ctgit_object_param (const Cgi self, const char* v, size_t v_sz, void* u_dat)
   if (!q->action)
     q->action = cwa_strndup ("object", SIZE_MAX);
 
+  /* TODO validate that v is a probably sha1 (<=40 chars, hex) */
+
   free (q->object);
   q->object = cwa_strndup (v, v_sz);
 }
@@ -119,6 +125,8 @@ ctgit_ref_param (const Cgi self, const char* v, size_t v_sz, void* u_dat)
   (void) self;
   struct ctgit_query* q = (struct ctgit_query*) u_dat;
 
+  /* TODO validate that v is a probably legal reference (alnum() || one of '_/.') */
+
   free (q->head);
   q->head = cwa_strndup (v, v_sz);
 }