<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
</head>
<body text="#000000" bgcolor="#FFFFFF">
<p><br>
</p>
<div class="moz-cite-prefix">On 12/3/2019 10:12 AM, Gunnar Mills
wrote:<br>
</div>
<blockquote type="cite"
cite="mid:5747d991-27b7-4bb1-8a25-f46d1de832e1@linux.vnet.ibm.com">
<br>
<blockquote type="cite">In BMC WebUI under "Download image file
from TFTP server" section,
<br>
we have text fields "TFTP Server IP address" and "File name".
<br>
"File name" doesn't take folders in path. Is this a bug or
expected behavior?
<br>
<br>
TFTP downloads work only if file is kept in root of tftp share.
<br>
<br>
</blockquote>
This is expected.
<br>
<a class="moz-txt-link-freetext" href="https://github.com/openbmc/phosphor-bmc-code-mgmt/blob/b0ce996ac60cf80487d71c3cdb7165d065079377/download_manager.cpp#L33">https://github.com/openbmc/phosphor-bmc-code-mgmt/blob/b0ce996ac60cf80487d71c3cdb7165d065079377/download_manager.cpp#L33</a>
<br>
</blockquote>
<p>As long as we continue to sanitize the local filename, I don't
see why we need to limit the source file name.<br>
Patrick, Adriana, Any objection? Remember why we wrote it this
way? I assume for simplicity..?<br>
<br>
Thanks, <br>
Gunnar<br>
<span class="c-message_attachment__text"
data-qa="message_attachment_text"><span dir="auto"></span></span></p>
</body>
</html>