Skip to content

Conversation

@thaJeztah
Copy link
Member

backport of #2819 for the 18.09 branch; cherry-pick was clean, no conflicts

@thaJeztah
Copy link
Member Author

ping @andrewhsu @dperny @andrey-ko PTAL

@thaJeztah thaJeztah force-pushed the 18.09_backport_fix_crash branch from 104f1b0 to eef2605 Compare February 11, 2019 10:57
@thaJeztah

This comment has been minimized.

@codecov
Copy link

codecov bot commented Feb 11, 2019

Codecov Report

Merging #2825 into bump_v18.09 will increase coverage by 0.14%.
The diff coverage is 0%.

@@               Coverage Diff               @@
##           bump_v18.09    #2825      +/-   ##
===============================================
+ Coverage        61.71%   61.85%   +0.14%     
===============================================
  Files              134      134              
  Lines            21868    21870       +2     
===============================================
+ Hits             13495    13528      +33     
+ Misses            6919     6880      -39     
- Partials          1454     1462       +8

@thaJeztah thaJeztah changed the title [18.09 backport] fix crash when there are attachments with null network [WIP][18.09 backport] fix crash when there are attachments with null network Feb 11, 2019
@thaJeztah
Copy link
Member Author

marking this "WIP" as we should have the fix merged on master first

Signed-off-by: Andrey Kolomentsev <[email protected]>
(cherry picked from commit 5be0152)
Signed-off-by: Sebastiaan van Stijn <[email protected]>
@thaJeztah thaJeztah force-pushed the 18.09_backport_fix_crash branch from eef2605 to 6ae0a2f Compare February 17, 2019 14:09
@thaJeztah
Copy link
Member Author

rebased to get the latest version of the patch, but #2819 isn't merged yet, so keeping the "WIP" prefix

@thaJeztah
Copy link
Member Author

closing, as 18.09 is EOL

@thaJeztah thaJeztah closed this Jun 5, 2021
@thaJeztah thaJeztah deleted the 18.09_backport_fix_crash branch June 5, 2021 21:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants