[Webkit-unassigned] [Bug 45113] FTP EPLF does not handle directory

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Sun Dec 19 04:01:35 PST 2021


https://bugs.webkit.org/show_bug.cgi?id=45113

--- Comment #20 from Pascal Abresch <nep-webkit at packageloss.eu> ---
Looking at the MacOS test results it seems there is a second problem, that of the timestamp display. The Haiku port has a patch to change the display of timestamps in the ftp code to use the OS timezone for display instead of gmttime.

How does the TestSuite deal with such situations? Is there a predictable timezone set for tests or would for example the python test have to change the send listing based on the current timezone?

And as a second question should the code to change the timestamp display be submitted in this same changeset?

The diff, for reference:
--- a/Source/WebCore/loader/FTPDirectoryParser.cpp
+++ b/Source/WebCore/loader/FTPDirectoryParser.cpp
@@ -156,7 +156,7 @@ FTPEntryType parseOneFTPLine(const char* line, ListState& state, ListResult& res
                 time_t t = static_cast<time_t>(seconds);

                 // FIXME: This code has the year 2038 bug
-                gmtime_r(&t, &result.modifiedTime);
+                localtime_r(&t, &result.modifiedTime);
                 result.modifiedTime.tm_year += 1900;
               }
             }

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20211219/3e367019/attachment.htm>


More information about the webkit-unassigned mailing list