Syscall param socketcall.recvfrom(buf)指向不可寻址的字节

时间:2015-05-16 09:53:26

标签: c++ sockets valgrind

运行此方法时,我在调用socket.getBytes

时从Valgrind收到错误
void Client::register(VMap::VType type, char *id)
{
    const int sizeOfType = sizeof(type);
    char *buffer = (char *) malloc(sizeOfType);
    memcpy(buffer, &type, sizeOfType);
    socket.send(buffer, sizeOfType);
    sleep(1);
    char *bufferRecv = (char *) malloc(sizeOfType);
    memset(bufferRecv, 0 ,sizeOfType);
    int size = socket.getBytes(bufferRecv);

    free(buffer);
    free(bufferRecv);
}


int Socket::getBytes(char *buf)
{
    int ret = ::recv(m_sock, buf, MAXRECV, 0);

    return ret;
}

错误是:

==30653== Syscall param socketcall.recvfrom(buf) points to unaddressable byte(s)
==30653==    at 0x58C214C: recv (recv.c:34)
==30653==    by 0x4E47B84: pippo::Socket::getBytes(char*) (Socket.cpp:115)
==30653==    by 0x4E48BAE: pippo::Rossi::register(pippo::sMap::Type, char*) (Rossi.cpp:80)
==30653==    by 0x4E48EEE: pippo::Rossi::initsMap(std::string) (Rossi.cpp:111)
==30653==    by 0x4E486B0: pippo::Rossi::Rossi(std::string, std::string, std::string, unsigned int) (Rossi.cpp:54)
==30653==    by 0x4E45884: test::TestRossi::testConstructor() (TestRossi.cpp:70)
==30653==    by 0x4E456D8: test::TestRossi::testBody() (TestRossi.cpp:48)
==30653==    by 0x5288D03: test::TestUnit::test() (TestUnit.cpp:42)
==30653==    by 0x5288513: test::TestRunner::run() (TestRunner.cpp:44)
==30653==    by 0x407F26: main (TestDUMBO.cpp:125)
==30653==  Address 0x67a9d64 is 0 bytes after a block of size 4 alloc'd
==30653==    at 0x4C2B6CD: malloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==30653==    by 0x4E48B21: pippo::Rossi::register(pippo::sMap::Type, char*) (Rossi.cpp:76)
==30653==    by 0x4E48EEE: pippo::Rossi::initsMap(std::string) (Rossi.cpp:111)
==30653==    by 0x4E486B0: pippo::Rossi::Rossi(std::string, std::string, std::string, unsigned int) (Rossi.cpp:54)
==30653==    by 0x4E45884: test::TestRossi::testConstructor() (TestRossi.cpp:70)
==30653==    by 0x4E456D8: test::TestRossi::testBody() (TestRossi.cpp:48)
==30653==    by 0x5288D03: test::TestUnit::test() (TestUnit.cpp:42)
==30653==    by 0x5288513: test::TestRunner::run() (TestRunner.cpp:44)
==30653==    by 0x407F26: main (TestDUMBO.cpp:125)

我真的不明白什么是错的。例如,如果我只是使用bufferRecvsprintf内写字,我就不会收到任何错误。

:: recv的原型如下:

/* Read N bytes into BUF from socket FD.
   Returns the number read or -1 for errors.

   This function is a cancellation point and therefore not marked with
   __THROW.  */
extern ssize_t recv (int __fd, void *__buf, size_t __n, int __flags);

编辑另一个信息:如果我像这样char bufferRecv[4];声明缓冲区,我不会从Valgrind那里得到任何错误。

1 个答案:

答案 0 :(得分:1)

对于有同样问题的其他人: 正如@alk所建议的,我只是按如下方式更改了getBytes方法:

int Socket::getBytes(char *buf, int sizeOfBuf)
{
    int ret = ::recv(m_sock, buf, sizeOfBuf, 0);


    return ret;
}