Receiving weird characters when using UDP

[Niely]

Hello

My code does not give any error, but it ain't really working like it should be.
I'm trying to send a simple message over UDP.

Code:

#include <iostream>
#include <string>
#include <SFML/System.hpp>
#include <SFML/Network.hpp>

using namespace std;

int main() {
        sf::UdpSocket socket;
        string receiver;
        string msg;
        socket.bind(55001);
       
        //Configure send options
        /*cout << "Send to: ";
        getline(cin, receiver);*/
        receiver = "MY ONLINE IP";
        cout << "Message: ";
        getline(cin, msg);

        //Send to myself (IPCONFIG)
        socket.send(msg.c_str(), msg.size() +1, receiver, 55001);

        //Receive from myself
        char buffer[1024];
        size_t received = 0;
        sf::IpAddress sender;
        unsigned short port;
       
        socket.receive(buffer, sizeof(buffer), received, sender, port);
        cout << endl << endl << "From: " << sender << " - " << buffer << endl;

        cin.ignore().get();
        return 0;
}


 

This code, is filling the rest of the buffer with weird characters.

Message: Test


From: 0.0.0.0 - ╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠
╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠
╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠
╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠
╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠
╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠
╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠
╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠
╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠
╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠
╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠
╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠
╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠
╠╠╠╠╠╠╠╠pï▬☺Test

How do I need to fix this?

Thanks for reading,

[eXpl0it3r]

The buffer array is uninitialized so it contains random characters. So the problem is not related to UDP at all, it's just that you use uninitialized memory.

Additionally don't use endl and don't use using namespace std;.
std::endl does also flush the stream which is not always what you want, just use "\n" instead and it might be slightly more expressive.
And by explicitly writing the namespace, you'll make your code a lot more readable, because at a glance everybody knows from where a function originates from.

[Niely]

If I initialize it, it just returns an empty value or with whatever I initialize it.

[Jesper Juhl]

Don't read more from the buffer than receive() put into it.

[Laurent]

buffer[received] = '\0';
std::cout << buffer;

// or

std::cout << std::string(buffer, buffer + received);

[Nexus]

buffer[received] = '\0';
std::cout << buffer;

That can be dangerous when received == sizeof(buffer).

[Laurent]

Then receive sizeof(buffer) - 1

[Nexus]

Yeah, until the next maintainer thinks "why? we have one byte left" and two months later, peculiar bugs appear at the other corner of the application

[Niely]

Tried both solutions, none worked.
Still getting something empty returned.

[Laurent]

Tried both solutions, none worked.

Full code + output please

[Niely]

#include <iostream>
#include <string>
#include <SFML/System.hpp>
#include <SFML/Network.hpp>

using namespace std;

int main() {
        sf::UdpSocket socket;
        string receiver;
        string msg;
        socket.bind(55001);
       
        //Configure send options
        /*cout << "Send to: ";
        getline(cin, receiver);*/
        receiver = "84.195.14.184";
        cout << "Message: ";
        getline(cin, msg);

        //Send to myself (IPCONFIG)
        socket.send(msg.c_str(), msg.size() +1, receiver, 55001);

        //Receive from myself
        char buffer[1024];
/*Also tried: char buffer[1024] = ""; & char buffer[1024] = {}; & ...*/
        size_t received = 0;
        sf::IpAddress sender;
        unsigned short port;
       
        socket.receive(buffer, sizeof(buffer), received, sender, port);
        buffer[received] = '\0';
        cout << "\n" << "\n" << "From: " << sender << " - " <<  buffer /*also tried only 'string(buffer, buffer + received)' without '\0'*/ << "\n";

        cin.ignore().get();
        return 0;
}


 

I can't declare my buffer like buffer[received], because I need it in my function 'send', and only send returns 'receive'.


Output: From: 0.0.0.0 -

[Laurent]

This is not the same output at all: no more garbage, it's completely empty. Print more information: the status returned by the receive function, and the value of the "received" variable. These are mandatory before trying to interpret the reception buffer.

[Niely]

#include <iostream>
#include <string>
#include <SFML/System.hpp>
#include <SFML/Network.hpp>

using namespace std;

int main() {
        sf::UdpSocket socket;
        string receiver;
        string msg;
        socket.bind(55001);

        //Configure send options
        /*cout << "Send to: ";
        getline(cin, receiver);*/
        receiver = "YOUR PUBLIC WI-FI IP";
        cout << "Message: ";
        getline(cin,msg);

        //Send to myself (IPCONFIG)
        int y = socket.send(msg.c_str(),msg.size() +1,receiver,55001);
        cout << "Send: " << y << endl;

        //Receive from myself
        char buffer[1024];
        size_t received = 0;
        sf::IpAddress sender;
        unsigned short port;

        int x = socket.receive(buffer,sizeof(buffer),received,sender,port);
        buffer[received] = '\0';
        cout << endl << endl << "From: " << sender << " - " <<  buffer << endl << "Receive: " << x;

        cin.ignore().get();
        return 0;
}
 

Output:

Message: Test
Send: 0


From: 0.0.0.0 -
Receive: 3

Send returns 0, receive returns 3.

[Laurent]

Now see what sf::Socket::Status it is, try to interpret, ...

[Niely]

I already tried looking into that. But status 3 is 'Error', an unknown error occurred.
How do I need to see my error in there?