ui-tree: put reverse path in title

[ps-cgit] / ui-shared.c
diff --git a/ui-shared.c b/ui-shared.c

index f6d38deb620de372456ea11d89eecfa6aa247f45..54bbde757951df30824484969276578f8c0a8650 100644 (file)
--- a/ui-shared.c
+++ b/ui-shared.c
@@ -54,14 +54,14 @@ const char *cgit_httpscheme(void)
                 return "http://";
  }
  
-const char *cgit_hosturl(void)
+char *cgit_hosturl(void)
  {
         if (ctx.env.http_host)
-               return ctx.env.http_host;
+               return xstrdup(ctx.env.http_host);
         if (!ctx.env.server_name)
                 return NULL;
         if (!ctx.env.server_port || atoi(ctx.env.server_port) == 80)
-               return ctx.env.server_name;
+               return xstrdup(ctx.env.server_name);
         return fmtalloc("%s:%s", ctx.env.server_name, ctx.env.server_port);
  }
  
@@ -525,7 +525,7 @@ void cgit_object_link(struct object *obj)
  {
         char *page, *shortrev, *fullrev, *name;
  
-       fullrev = sha1_to_hex(obj->sha1);
+       fullrev = oid_to_hex(&obj->oid);
         shortrev = xstrdup(fullrev);
         shortrev[10] = '\0';
         if (obj->type == OBJ_COMMIT) {
@@ -692,9 +692,11 @@ void cgit_print_http_headers(void)
                 htmlf("Content-Type: %s\n", ctx.page.mimetype);
         if (ctx.page.size)
                 htmlf("Content-Length: %zd\n", ctx.page.size);
-       if (ctx.page.filename)
-               htmlf("Content-Disposition: inline; filename=\"%s\"\n",
-                     ctx.page.filename);
+       if (ctx.page.filename) {
+               html("Content-Disposition: inline; filename=\"");
+               html_header_arg_in_quotes(ctx.page.filename);
+               html("\"\n");
+       }
         if (!ctx.env.authenticated)
                 html("Cache-Control: no-cache, no-store\n");
         htmlf("Last-Modified: %s\n", http_date(ctx.page.modified));
@@ -709,7 +711,9 @@ void cgit_print_http_headers(void)
  void cgit_redirect(const char *url, bool permanent)
  {
         htmlf("Status: %d %s\n", permanent ? 301 : 302, permanent ? "Moved" : "Found");
-       htmlf("Location: %s\n\n", url);
+       html("Location: ");
+       html_url_path(url);
+       html("\n\n");
         exit(0);
  }
  
@@ -730,7 +734,7 @@ void cgit_print_docstart(void)
                 return;
         }
  
-       const char *host = cgit_hosturl();
+       char *host = cgit_hosturl();
         html(cgit_doctype);
         html("<html xmlns='http://www.w3.org/1999/xhtml' xml:lang='en' lang='en'>\n");
         html("<head>\n");
@@ -755,7 +759,7 @@ void cgit_print_docstart(void)
  
                 html("<link rel='alternate' title='Atom feed' href='");
                 html(cgit_httpscheme());
-               html_attr(cgit_hosturl());
+               html_attr(host);
                 fileurl = cgit_fileurl(ctx.repo->url, "atom", ctx.qry.vpath,
                                        sb.buf);
                 html_attr(fileurl);
@@ -771,6 +775,7 @@ void cgit_print_docstart(void)
         html("<body>\n");
         if (ctx.cfg.header)
                 html_include(ctx.cfg.header);
+       free(host);
  }
  
  void cgit_print_docend(void)
@@ -888,6 +893,9 @@ void cgit_add_hidden_formfields(int incl_head, int incl_search,
  
  static const char *hc(const char *page)
  {
+       if (!ctx.qry.page)
+               return NULL;
+
         return strcmp(ctx.qry.page, page) ? NULL : "active";
  }