leeexin/Velix-MPV-Core
1
0 Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

m_property: fix playlist property parsing

Browse Source 
Disallow silly things like ${playlist//} and trailing slashes
${playlist/0/}.
This commit is contained in:
Kacper Michajłow authored and Dudemanguy committed 2025-02-08 02:01:52 +00:00
1 parent 3c1ddc411c
commit 95019fc256
1 file changed
+4 -6
Download Patch File Download Diff File Expand all files Collapse all files
options/m_property.c
+4 -6
View File
@@ -617,19 +617,17 @@ int m_property_read_list(int action, void *arg, int count,
return M_PROPERTY_NOT_IMPLEMENTED; return M_PROPERTY_NOT_IMPLEMENTED;
} }
// This is expected of the form "123" or "123/rest" // This is expected of the form "123" or "123/rest"
char *next = strchr(ka->key, '/'); char *end;
char *end = NULL;
const char *key_end = ka->key + strlen(ka->key);
long int item = strtol(ka->key, &end, 10); long int item = strtol(ka->key, &end, 10);
// not a number, trailing characters, etc. // not a number, trailing characters, etc.
if ((end != key_end || ka->key == key_end) && end != next) if (end == ka->key || (end[0] == '/' && !end[1]))
return M_PROPERTY_UNKNOWN; return M_PROPERTY_UNKNOWN;
if (item < 0 || item >= count) if (item < 0 || item >= count)
return M_PROPERTY_UNKNOWN; return M_PROPERTY_UNKNOWN;
if (next) { if (*end) {
// Sub-path // Sub-path
struct m_property_action_arg n_ka = *ka; struct m_property_action_arg n_ka = *ka;
n_ka.key = next + 1; n_ka.key = end + 1;
return get_item(item, M_PROPERTY_KEY_ACTION, &n_ka, ctx); return get_item(item, M_PROPERTY_KEY_ACTION, &n_ka, ctx);
} else { } else {
// Direct query // Direct query