
<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;"><pre>Hi Klaus,</pre><pre><br></pre><pre>> It's funny, however, that it works (i.e., didn't crash for me on</pre><pre>> the zero-length path scenario)</pre><pre><br></pre><pre>Yep, from reading your v1 patch, it looked like there was no way for</pre><pre>grub2_parse_file to return a NULL ->path, so I don't think you were</pre><pre>hitting that case.</pre><pre><br></pre><pre>> > Regardless, in the zero-length path case, -> path is set to NULL,</pre><pre>> > ->dev is not set either, so this function still returns NULL. Are</pre><pre>> > you sure that this fixes the issue in the way that you expect?</pre><pre>> </pre><pre>> I'm not sure if '()/something' is legal, even less '()'. For</pre><pre>> consistency,</pre><pre><br></pre><pre>This is regarding an empty path specifier, not an empty device. It looks</pre><pre>like you've addressed that in v3 though.</pre><pre><br></pre><pre><br></pre><pre>> can we settle for both dev as well as path can EITHER be zero-length</pre><pre>> (but not both at the same time)?</pre><pre><br></pre><pre>Yep, I think that's a good approach.</pre><pre><br></pre><pre>Cheers,</pre><pre><br></pre><pre><br></pre><pre>Jeremy</pre></body></html>