mirror of
https://github.com/systemd/systemd.git
synced 2024-11-08 02:57:16 +03:00
import: hash URL in paths if they are too long
https://bugzilla.redhat.com/show_bug.cgi?id=1266775
This commit is contained in:
parent
1f31211da9
commit
9818005de6
@ -31,8 +31,10 @@
|
||||
#include "pull-common.h"
|
||||
#include "process-util.h"
|
||||
#include "signal-util.h"
|
||||
#include "siphash24.h"
|
||||
|
||||
#define FILENAME_ESCAPE "/.#\"\'"
|
||||
#define HASH_URL_THRESHOLD_LENGTH (_POSIX_PATH_MAX - 16)
|
||||
|
||||
int pull_find_old_etags(
|
||||
const char *url,
|
||||
@ -149,8 +151,21 @@ int pull_make_local_copy(const char *final, const char *image_root, const char *
|
||||
return 0;
|
||||
}
|
||||
|
||||
static int hash_url(const char *url, char **ret) {
|
||||
uint64_t h;
|
||||
static const sd_id128_t k = SD_ID128_ARRAY(df,89,16,87,01,cc,42,30,98,ab,4a,19,a6,a5,63,4f);
|
||||
|
||||
assert(url);
|
||||
|
||||
siphash24((uint8_t *) &h, url, strlen(url), k.bytes);
|
||||
if (asprintf(ret, "%"PRIx64, h) < 0)
|
||||
return -ENOMEM;
|
||||
|
||||
return 0;
|
||||
}
|
||||
|
||||
int pull_make_path(const char *url, const char *etag, const char *image_root, const char *prefix, const char *suffix, char **ret) {
|
||||
_cleanup_free_ char *escaped_url = NULL;
|
||||
_cleanup_free_ char *escaped_url = NULL, *escaped_etag = NULL;
|
||||
char *path;
|
||||
|
||||
assert(url);
|
||||
@ -164,18 +179,35 @@ int pull_make_path(const char *url, const char *etag, const char *image_root, co
|
||||
return -ENOMEM;
|
||||
|
||||
if (etag) {
|
||||
_cleanup_free_ char *escaped_etag = NULL;
|
||||
|
||||
escaped_etag = xescape(etag, FILENAME_ESCAPE);
|
||||
if (!escaped_etag)
|
||||
return -ENOMEM;
|
||||
}
|
||||
|
||||
path = strjoin(image_root, "/", strempty(prefix), escaped_url, ".", escaped_etag, strempty(suffix), NULL);
|
||||
} else
|
||||
path = strjoin(image_root, "/", strempty(prefix), escaped_url, strempty(suffix), NULL);
|
||||
path = strjoin(image_root, "/", strempty(prefix), escaped_url, escaped_etag ? "." : "",
|
||||
strempty(escaped_etag), strempty(suffix), NULL);
|
||||
if (!path)
|
||||
return -ENOMEM;
|
||||
|
||||
/* URLs might make the path longer than the maximum allowed length for a file name.
|
||||
* When that happens, a URL hash is used instead. Paths returned by this function
|
||||
* can be later used with tempfn_random() which adds 16 bytes to the resulting name. */
|
||||
if (strlen(path) >= HASH_URL_THRESHOLD_LENGTH) {
|
||||
_cleanup_free_ char *hash = NULL;
|
||||
int r;
|
||||
|
||||
free(path);
|
||||
|
||||
r = hash_url(url, &hash);
|
||||
if (r < 0)
|
||||
return r;
|
||||
|
||||
path = strjoin(image_root, "/", strempty(prefix), hash, escaped_etag ? "." : "",
|
||||
strempty(escaped_etag), strempty(suffix), NULL);
|
||||
if (!path)
|
||||
return -ENOMEM;
|
||||
}
|
||||
|
||||
*ret = path;
|
||||
return 0;
|
||||
}
|
||||
|
Loading…
Reference in New Issue
Block a user