Fossil SCM
Fix another problem with the needEscape computation in blob_append_escaped_arg()
Commit
9eea719af68932ad5a9930f86e4656b249eae995530396b78308969882f309be
Parent
49ae1785a6149c8…
1 file changed
+1
-1
+1
-1
| --- src/blob.c | ||
| +++ src/blob.c | ||
| @@ -1202,11 +1202,11 @@ | ||
| 1202 | 1202 | blob_token(pBlob, &bad); |
| 1203 | 1203 | fossil_fatal("the [%s] argument to the \"%s\" command contains " |
| 1204 | 1204 | "a character (ascii 0x%02x) that is a security risk", |
| 1205 | 1205 | zIn, blob_str(&bad), c); |
| 1206 | 1206 | } |
| 1207 | - if( !needEscape && !fossil_isspace(c) && c!='/' && c!='.' && c!='_' ){ | |
| 1207 | + if( !needEscape && !fossil_isalnum(c) && c!='/' && c!='.' && c!='_' ){ | |
| 1208 | 1208 | needEscape = 1; |
| 1209 | 1209 | } |
| 1210 | 1210 | } |
| 1211 | 1211 | if( n>0 && !fossil_isspace(z[n-1]) ){ |
| 1212 | 1212 | blob_append(pBlob, " ", 1); |
| 1213 | 1213 |
| --- src/blob.c | |
| +++ src/blob.c | |
| @@ -1202,11 +1202,11 @@ | |
| 1202 | blob_token(pBlob, &bad); |
| 1203 | fossil_fatal("the [%s] argument to the \"%s\" command contains " |
| 1204 | "a character (ascii 0x%02x) that is a security risk", |
| 1205 | zIn, blob_str(&bad), c); |
| 1206 | } |
| 1207 | if( !needEscape && !fossil_isspace(c) && c!='/' && c!='.' && c!='_' ){ |
| 1208 | needEscape = 1; |
| 1209 | } |
| 1210 | } |
| 1211 | if( n>0 && !fossil_isspace(z[n-1]) ){ |
| 1212 | blob_append(pBlob, " ", 1); |
| 1213 |
| --- src/blob.c | |
| +++ src/blob.c | |
| @@ -1202,11 +1202,11 @@ | |
| 1202 | blob_token(pBlob, &bad); |
| 1203 | fossil_fatal("the [%s] argument to the \"%s\" command contains " |
| 1204 | "a character (ascii 0x%02x) that is a security risk", |
| 1205 | zIn, blob_str(&bad), c); |
| 1206 | } |
| 1207 | if( !needEscape && !fossil_isalnum(c) && c!='/' && c!='.' && c!='_' ){ |
| 1208 | needEscape = 1; |
| 1209 | } |
| 1210 | } |
| 1211 | if( n>0 && !fossil_isspace(z[n-1]) ){ |
| 1212 | blob_append(pBlob, " ", 1); |
| 1213 |