Possible incorrect logical operator used in if statement in OnlineSessionInterfaceNull.cpp

A user pointed out a possible mistaken use of the && operator instead of the || operator in an if statement in source code. In the FOnlineSessionNull::GetResolvedConnectString() function in OnlineSessionInterfaceNull.cpp, a default beacon listen port value is set but never used if no port was actually specified.

1. Open OnlineSessionInterfaceNull.cpp.
2. Locate the function FOnlineSessionNull::GetResolvedConnectString(const class FOnlineSessionSearchResult& SearchResult, FName PortType, FString& ConnectInfo)
   • Line 739 in CL 2955029 in Dev-Main.

RESULT:
Line 749 reads if (SearchResult.Session.SessionSettings.Get(SETTING_BEACONPORT, BeaconListenPort) && BeaconListenPort > 0).

EXPECTED:
The user suggested that the || operator would be better used here so that the default value for BeaconListenPort is used if a value has not been specifically set in the SETTING_BEACONPORT key.

"Session Disconnected" UEFN

How to delete some elements correctly when deleting an array loop?

Do I need to call Destory method to remove components from Actor when Destory method to remove Actor?

How does TextureRenderTarget2D get TArray<uint8> type data?

What is the difference between Camera and CineCamera?

UMG RichText not appear image when packaged

How do I set a material as a post-processing material?

Why does the REMOVE method of map container remove elements have memory leaks?

How does TArray loop correctly remove elements in blueprints?

How to assign a value to a member of UMG's UMaterialInterface pointer type in C++?