-
Notifications
You must be signed in to change notification settings - Fork 7.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
stop corrupting memory in libpsl-native SetDate #6881
Conversation
return result; | ||
} | ||
|
||
return settimeofday(&tv, NULL); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why do we use settimeofday if we don't use second parameter (timezone)?
Could we use stime? In the case we have to adjust only one parameter time_t
.
Second question do this parameter adjustements work correctly for ARM reverse byte order?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There are known issues there this isn't working on ARM. @dantraMSFT mentioned this to me.
Feel free to file an issue to switch to stime, although the better fix is probably to include the timezone. This fix is scoped to stop corrupting memory.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems the timezone can be a issue too. We should look this in depth. Will push new Issue to discuss better solution.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tracking issue #6962
LGTM to get this in Preview3. |
PR Summary
stop corrupting memory in libpsl-native SetDate
Addresses #6872 - the first step
PR Checklist
.h
,.cpp
,.cs
,.ps1
and.psm1
files have the correct copyright headerWIP:
to the beginning of the title and remove the prefix when the PR is ready.[feature]
if the change is significant or affects feature tests