Skip to content

Correct NullPointerException on UseDataDictionary=N and FIXT session #775

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 2 commits into from
Mar 14, 2024
Merged

Correct NullPointerException on UseDataDictionary=N and FIXT session #775

merged 2 commits into from
Mar 14, 2024

Conversation

@wajncn
Copy link

@wajncn wajncn commented Mar 14, 2024 •
edited by chrjohn
Loading

UseDataDictionary=N , dataDictionaryProvider is null
Fixes #776

@wajncn
Copy link
Author

wajncn commented Mar 14, 2024

if config UseDataDictionary=N ,DataDictionaryProvider is null. run after throw null point exception.
in File Session.java 2167Line [quickfix.DataDictionaryProvider#getSessionDataDictionary].

@wajncn wajncn mentioned this pull request Mar 14, 2024
Closed
@chrjohn
Copy link
Member

chrjohn commented Mar 14, 2024

Hi @wajncn
Thanks for the PR. Good catch 👍

@chrjohn chrjohn added this to the QFJ 3.0.0 milestone Mar 14, 2024
@chrjohn chrjohn changed the title fix UseDataDictionary=N dataDictionaryProvider is null Correct NullPointerException on UseDataDictionary=N and FIXT session Mar 14, 2024
@chrjohn chrjohn merged commit d7c9fe5 into quickfix-j:master Mar 14, 2024
@wajncn wajncn deleted the fix-npe branch March 15, 2024 02:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

QFJ 3.0.0

Development

Successfully merging this pull request may close these issues.

NullPointerException when using UseDataDictionary=N and FIXT session

2 participants

@wajncn @chrjohn