public inbox for linux-tegra@vger.kernel.org
 help / color / mirror / Atom feed
* [cbootimage] Allow @ in paths consumed by cbootimage
@ 2016-03-10 17:34 Patrick Georgi
       [not found] ` <CAE-gjdUeMdnNcPOZJpCz1_V7FKAh0aHMpqkhkkWpA4XpjPOo4Q-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
  0 siblings, 1 reply; 2+ messages in thread
From: Patrick Georgi @ 2016-03-10 17:34 UTC (permalink / raw)
  To: linux-tegra-u79uwXL29TY76Z2rM5mHXA

Signed-off-by: Patrick Georgi <pgeorgi-hpIqsD4AKlfQT0dZR+AlfA@public.gmane.org>

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--;


-- 
Google Germany GmbH, ABC-Str. 19, 20354 Hamburg
Registergericht und -nummer: Hamburg, HRB 86891, Sitz der Gesellschaft: Hamburg
Geschäftsführer: Matthew Scott Sucherman, Paul Terence Manicle

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [cbootimage] Allow @ in paths consumed by cbootimage
       [not found] ` <CAE-gjdUeMdnNcPOZJpCz1_V7FKAh0aHMpqkhkkWpA4XpjPOo4Q-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
@ 2016-03-10 23:53   ` Stephen Warren
  0 siblings, 0 replies; 2+ messages in thread
From: Stephen Warren @ 2016-03-10 23:53 UTC (permalink / raw)
  To: Patrick Georgi; +Cc: linux-tegra-u79uwXL29TY76Z2rM5mHXA

On 03/10/2016 10:34 AM, Patrick Georgi wrote:
> Signed-off-by: Patrick Georgi <pgeorgi-hpIqsD4AKlfQT0dZR+AlfA@public.gmane.org>

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.

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2016-03-10 23:53 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-03-10 17:34 [cbootimage] Allow @ in paths consumed by cbootimage Patrick Georgi
     [not found] ` <CAE-gjdUeMdnNcPOZJpCz1_V7FKAh0aHMpqkhkkWpA4XpjPOo4Q-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2016-03-10 23:53   ` Stephen Warren

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox