author | Lars Hjemli <hjemli@gmail.com> | 2007-05-20 20:09:55 (UTC) |
---|---|---|
committer | Lars Hjemli <hjemli@gmail.com> | 2007-05-20 20:09:55 (UTC) |
commit | dc3ac3f76077c5d612d42e8beb4878e43acfc58a (patch) (unidiff) | |
tree | dfb996c0ce9833841578e1f0accbb5a387c01237 /cache.c | |
parent | 977a0b173df6fe1a4d362fe4c70f9badff1fd46c (diff) | |
parent | 3de63b264c36888dfd42dfdf3fc0aad4ce0c2b5c (diff) | |
download | cgit-dc3ac3f76077c5d612d42e8beb4878e43acfc58a.zip cgit-dc3ac3f76077c5d612d42e8beb4878e43acfc58a.tar.gz cgit-dc3ac3f76077c5d612d42e8beb4878e43acfc58a.tar.bz2 |
Merge branch 'virtual-url'
* virtual-url:
Don't be fooled by trailing '/' in url-parameter
cache_safe_filename() needs more buffers
Enable url=value querystring parameter
Add lookup-function for valid repo commands
Move cgit_get_repoinfo into shared.c
-rw-r--r-- | cache.c | 22 |
1 files changed, 15 insertions, 7 deletions
@@ -7,28 +7,33 @@ | |||
7 | */ | 7 | */ |
8 | 8 | ||
9 | #include "cgit.h" | 9 | #include "cgit.h" |
10 | 10 | ||
11 | const int NOLOCK = -1; | 11 | const int NOLOCK = -1; |
12 | 12 | ||
13 | char *cache_safe_filename(const char *unsafe) | 13 | char *cache_safe_filename(const char *unsafe) |
14 | { | 14 | { |
15 | static char buf[PATH_MAX]; | 15 | static char buf[4][PATH_MAX]; |
16 | char *s = buf; | 16 | static int bufidx; |
17 | char *s; | ||
17 | char c; | 18 | char c; |
18 | 19 | ||
20 | bufidx++; | ||
21 | bufidx &= 3; | ||
22 | s = buf[bufidx]; | ||
23 | |||
19 | while(unsafe && (c = *unsafe++) != 0) { | 24 | while(unsafe && (c = *unsafe++) != 0) { |
20 | if (c == '/' || c == ' ' || c == '&' || c == '|' || | 25 | if (c == '/' || c == ' ' || c == '&' || c == '|' || |
21 | c == '>' || c == '<' || c == '.') | 26 | c == '>' || c == '<' || c == '.') |
22 | c = '_'; | 27 | c = '_'; |
23 | *s++ = c; | 28 | *s++ = c; |
24 | } | 29 | } |
25 | *s = '\0'; | 30 | *s = '\0'; |
26 | return buf; | 31 | return buf[bufidx]; |
27 | } | 32 | } |
28 | 33 | ||
29 | int cache_exist(struct cacheitem *item) | 34 | int cache_exist(struct cacheitem *item) |
30 | { | 35 | { |
31 | if (stat(item->name, &item->st)) { | 36 | if (stat(item->name, &item->st)) { |
32 | item->st.st_mtime = 0; | 37 | item->st.st_mtime = 0; |
33 | return 0; | 38 | return 0; |
34 | } | 39 | } |
@@ -38,25 +43,28 @@ int cache_exist(struct cacheitem *item) | |||
38 | int cache_create_dirs() | 43 | int cache_create_dirs() |
39 | { | 44 | { |
40 | char *path; | 45 | char *path; |
41 | 46 | ||
42 | path = fmt("%s", cgit_cache_root); | 47 | path = fmt("%s", cgit_cache_root); |
43 | if (mkdir(path, S_IRWXU) && errno!=EEXIST) | 48 | if (mkdir(path, S_IRWXU) && errno!=EEXIST) |
44 | return 0; | 49 | return 0; |
45 | 50 | ||
46 | if (!cgit_query_repo) | 51 | if (!cgit_repo) |
47 | return 0; | 52 | return 0; |
48 | 53 | ||
49 | path = fmt("%s/%s", cgit_cache_root, cgit_query_repo); | 54 | path = fmt("%s/%s", cgit_cache_root, |
55 | cache_safe_filename(cgit_repo->url)); | ||
56 | |||
50 | if (mkdir(path, S_IRWXU) && errno!=EEXIST) | 57 | if (mkdir(path, S_IRWXU) && errno!=EEXIST) |
51 | return 0; | 58 | return 0; |
52 | 59 | ||
53 | if (cgit_query_page) { | 60 | if (cgit_query_page) { |
54 | path = fmt("%s/%s/%s", cgit_cache_root, cgit_query_repo, | 61 | path = fmt("%s/%s/%s", cgit_cache_root, |
62 | cache_safe_filename(cgit_repo->url), | ||
55 | cgit_query_page); | 63 | cgit_query_page); |
56 | if (mkdir(path, S_IRWXU) && errno!=EEXIST) | 64 | if (mkdir(path, S_IRWXU) && errno!=EEXIST) |
57 | return 0; | 65 | return 0; |
58 | } | 66 | } |
59 | return 1; | 67 | return 1; |
60 | } | 68 | } |
61 | 69 | ||
62 | int cache_refill_overdue(const char *lockfile) | 70 | int cache_refill_overdue(const char *lockfile) |