diff mbox

[cbootimage] Allow @ in paths consumed by cbootimage

Message ID CAE-gjdUeMdnNcPOZJpCz1_V7FKAh0aHMpqkhkkWpA4XpjPOo4Q@mail.gmail.com
State Deferred
Delegated to: Stephen Warren
Headers show

Commit Message

Patrick Georgi March 10, 2016, 5:34 p.m. UTC 
Signed-off-by: Patrick Georgi <pgeorgi@google.com>

Comments

Stephen Warren March 10, 2016, 11:53 p.m. UTC | #1 
On 03/10/2016 10:34 AM, Patrick Georgi wrote:
> Signed-off-by: Patrick Georgi <pgeorgi@google.com>

This patch is corrupted; all the TABs seem to have been replaced with 
spaces. Perhaps you pasted the patch into an email editor instead on 
using git send-email?

This patch makes me dislike the current config file parsing code; I have 
no idea why it works like it does rather than just using strsep(..., 
",") to split up the input lines. Still, that's a pre-existing issue and 
nothing to do with your patch.
--
To unsubscribe from this list: send the line "unsubscribe linux-tegra" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git src/parse.c src/parse.c
index 6f37dad..39d746c 100644
--- src/parse.c
+++ src/parse.c
@@ -249,7 +249,7 @@  parse_filename(char *str, char *name, int chars_remaining)
         * Check if the filename buffer is out of space, preserving one
         * character to null terminate the string.
         */
-       while (isalnum(*str) || strchr("\\/~_-+:.", *str)) {
+       while (isalnum(*str) || strchr("\\/~_-+:.@", *str)) {

                chars_remaining--;