-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Why GeoLite2 placed in %APPDATA%? #2438
Labels
Comments
It is placed in app directory if SystemInformer.exe.settings.xml is present in app directory. |
ofc, just placed .settings in app directory |
how it confirigure path: systeminformer/plugins/NetworkTools/country.c Line 192 in 0815535
|
and there is memory leak: PPH_STRING PhGetApplicationDirectoryFileName(
_In_ PCPH_STRINGREF FileName,
_In_ BOOLEAN NativeFileName
)
{
PPH_STRING applicationFileName = NULL;
PPH_STRING applicationDirectory;
if (NativeFileName)
applicationDirectory = PhGetApplicationDirectory();
else
applicationDirectory = PhGetApplicationDirectoryWin32();
if (applicationDirectory)
{
applicationFileName = PhConcatStringRef2(&applicationDirectory->sr, FileName);
PhReferenceObject(applicationDirectory); // THERE IS DEREFERENCE REQUIRED, NOT REFERENCE!
}
return applicationFileName;
} |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Brief description of your issue
I use portable System Informer, it should placed in application directory!
Steps to reproduce (optional)
No response
Expected behavior (optional)
No response
Actual behavior (optional)
No response
Environment (optional)
The text was updated successfully, but these errors were encountered: