|
Bugzilla – Full Text Bug Listing |
| Summary: | m_windowOffsetsMap.end () dereference in TraceFadingLossModel::DoCalcRxPowerSpectralDensity | ||
|---|---|---|---|
| Product: | ns-3 | Reporter: | Alexander Krotov <krotov> |
| Component: | lte | Assignee: | Biljana Bojović <bbojovic> |
| Status: | RESOLVED FIXED | ||
| Severity: | major | CC: | ns-bugs, tomh, tommaso.pecorella |
| Priority: | P5 | ||
| Version: | ns-3-dev | ||
| Hardware: | All | ||
| OS: | All | ||
| Attachments: |
Proposed patch
Proposed patch |
||
Created attachment 2610 [details]
Proposed patch
forgot ".first"
Hi Alexander, thank you very much for reporting the bug and providing the patch for it. I have checked it and yes, ifOff needs to be initialized in both branches "if" and "else", since it is used later in the code. I am fine with the patch. Can you please commit/push it to ns-3-dev? Biljana Pushed in changeset 6ce2f4e01995 (In reply to Alexander Krotov from comment #3) > Pushed in changeset 6ce2f4e01995 Thanks! |
Created attachment 2609 [details] Proposed patch If m_windowOffsetsMap.find (mobilityPair) returns m_windowOffsetsMap.end (), it is stored into itOff. Later itOff is dereferenced to calculate index. To avoid dereferencing m_windowOffsetsMap.end (), itOff should be updated with a reference to newely inserted element.