> Am 11.07.2018 um 22:54 schrieb Joe Hershberger <joe.hershber...@ni.com>:
> 
> Hey Alex,
> 
>> On Thu, Jul 5, 2018 at 12:27 PM, Joe Hershberger <joe.hershber...@ni.com> 
>> wrote:
>>> On Thu, Jul 5, 2018 at 6:49 AM, Alexander Graf <ag...@suse.de> wrote:
>>>> On 07/04/2018 06:18 PM, Joe Hershberger wrote:
>>>> 
>>>>> On Wed, Jul 4, 2018 at 4:25 AM, Alexander Graf <ag...@suse.de> wrote:
>>>>> 
>>>>>> On 07/04/2018 02:36 AM, Joe Hershberger wrote:
>>>>>> 
>>>>>> Rather than crashing, check the src ptr and set dst to empty string.
>>>>>> 
>>>>>> Signed-off-by: Joe Hershberger <joe.hershber...@ni.com>
>>>>> 
>>>>> 
>>>>> Wouldn't it make more sense to check for the existence outside at the
>>>>> caller's side? That way it's much easier to see what really is happening.
>>>> 
>>>> It's much easier to allow NULL so that we can directly pass the return
>>>> result of getenv().
>>> 
>>> 
>>> I know, and I see how it looks insanely smart and simple today. Until you
>>> realize that the amazing "copy_filename" function doesn't touch the target
>>> at all if it gets passed in NULL. And all of that implicitly. So implicitly
>>> it will leave the old value in the filename if nothing is set in env.
>> 
>> I think you are mis-reading the code. If src is NULL, it will set
>> dst[0] = '\0';  I think the behavior is quite reasonable.
> 
> Do you have any outstanding issues with this?

Nope, your call :)

Alex


_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Reply via email to