FedRawDataInputSource: warning: 'newChunk' may be used uninitialized in this function

[iarspider]

A warning is issued in DBG_X IB CMSSW_13_1_DBG_X_2023-03-02-2300:

.../src/EventFilter/Utilities/src/FedRawDataInputSource.cc: In member function 'readSupervisor':
  .../src/EventFilter/Utilities/interface/FedRawDataInputSource.h:202:13: warning: 'newChunk' may be used uninitialized in this function [-Wmaybe-uninitialized]
   202 |     offset_ = newOffset;
      |             ^
.../src/EventFilter/Utilities/src/FedRawDataInputSource.cc:1205:21: note: 'newChunk' was declared here
 1205 |         InputChunk* newChunk;
      |                     ^

Full log: link

[cmsbuild]

A new Issue was created by @iarspider .

@Dr15Jones, @perrotta, @dpiparo, @rappoccio, @makortel, @smuzaffar can you please review it and eventually sign/assign? Thanks.

cms-bot commands are listed here

[makortel]

assign daq

[cmsbuild]

New categories assigned: daq

@smorovic,@emeschi you have been requested to review this Pull request/Issue and eventually sign? Thanks

[smorovic]

This is in the single-buffer mode that we don't use (and in fact will go away in the new input source for run3+phase2 that is in development).
I will prepare a patch to intialize variable and add checks to to avoid potential crash here.

[makortel]

Fixed in #40953

[makortel]

I suppose this issue could be closed now. @smorovic could you sign it?

[smorovic]

+1
yes, it was merged.

[cmsbuild]

This issue is fully signed and ready to be closed.

[makortel]

@cmsbuild, please close