mORMot Open Source

tbo

Member

Registered: 2015-04-20

Posts: 353

Issue: TSynLogFileView.Select

mORMot2, Commit 7264 (fedb2ee), Delphi 12, 32Bit
Unit mormot.core.log

Line 7756 should be changed. It should be checked that (fThreads[Index] > 0), otherwise the call to function GetBitPtr could run into nothing.

function TSynLogFileView.Select(aRow: integer): integer;
...
    for i := 0 to Count - 1 do
    begin
      if fLevels[i] in fEvents then
      begin
        if (fThreads = nil)
          or GetBitPtr(pointer(fThreadSelected), fThreads[i] - 1) then  // <-- here
        begin

With best regards
Thomas

ab

Administrator

From: France

Registered: 2010-06-21

Posts: 14,686

Website

Re: Issue: TSynLogFileView.Select

I don't see when fThreads[] could be = 0, unless the log file input is corrupted / was not generated by TSynLog.
Anyway, we should better be prepared for the worst.

Please try https://github.com/synopse/mORMot2/commit/6f368fe1

tbo

Member

Registered: 2015-04-20

Posts: 353

Re: Issue: TSynLogFileView.Select

I don't see when fThreads[] could be = 0, unless the log file input is corrupted / was not generated by TSynLog.

If remote logging is used (TRestHttpsClient.CreateForRemoteLogging and LogView), this may be the case.

With best regards
Thomas

tbo

Member

Registered: 2015-04-20

Posts: 353

Re: Issue: TSynLogFileView.Select

Please try https://github.com/synopse/mORMot2/commit/6f368fe1

Commit 7267 tested and it works. Thank you very much and have a nice weekend.

With best regards
Thomas