Issue Details (XML | Word | Printable)

Key: CORE-5294
Type: Bug Bug
Status: Resolved Resolved
Resolution: Fixed
Priority: Major Major
Assignee: Alexander Peshkov
Reporter: Pavel Zotov
Votes: 0
Watchers: 2
Operations

If you were logged in you would be able to see more operations.
Firebird Core

Memory leak when use SHOW GRANTS on new empty database (3.0.1 & 4.0; SS & SC)

Created: 27/Jun/16 09:45 PM   Updated: 01/Aug/16 09:57 AM
Component/s: None
Affects Version/s: 4.0 Initial, 3.0.0
Fix Version/s: 3.0.1, 4.0 Alpha 1

File Attachments: 1. File firebird.conf.files.7z (10 kB)
2. File memory-leak-when-use-show-grants-in-FB_301-or-FB_400.7z (162 kB)


QA Status: Cannot be tested
Test Details:
No way to detect memory leak using FB monitoring; it seems that Python 'psutil' package could help but it's not supplied in standard Python kit.


 Description  « Hide
1) Run ProcessExplorer or similar tool for monitoring process resources

2) create empty database, add it to databases.conf (aliases.conf for 2.5)

3) Run following .sql script several times:
===
set wng off;
set list on;
set echo on;

connect 'localhost:e25' user sysdba password 'masterke';
show grants;
quit;
===

Sample of batch for this ('fbc' variable stored path to different FB binaries on the same host ):
===
@echo off
set fbc=C:\MIX\firebird\fb25sS\bin
::set fbc=C:\MIX\firebird\fb25\bin
::set fbc=C:\MIX\firebird\fb30
::set fbc=C:\MIX\firebird\fb30sC
::set fbc=C:\MIX\firebird\fb40
::set fbc=C:\MIX\firebird\fb40sc

for /l %%i in (1,1,10) do (
  echo %%i
  %fbc%\isql -q -i memotrouble.sql 1>memotrouble.log 2>&1
)
===

On WI-T4.0.0.267 and WI-V3.0.1.32542 you will find noticable memory leak for FB process (I've tested bothSS and SC).
No such trouble on WI-V2.5.6.27017 SC or SS.

Please see screenshots in attached .7z file.


 All   Comments   Change History   Subversion Commits      Sort Order: Ascending order - Click to sort in descending order
Pavel Zotov added a comment - 27/Jun/16 09:54 PM
PS. All used firebird.conf files see in attached "firebird.conf.files.7z"

Alexander Peshkov added a comment - 30/Jun/16 04:18 PM
Confirmed on linux with default configs