Valgrind beschwert sich auf string.append (string)

Question

Ich versuche, eine Zeichenfolge an eine andere anzuhängen. Ich deklariere zwei globale String-Variablen -

Dann habe ich eine Funktion zum Abrufen von Befehlszeilenargumenten. Wann immer ein Benutzer einen Dateinamen in ein Befehlszeilenargument eingibt, sollte er im Ruhezustand gespeichert werden, und dann wird rest an grid_filename angehängt.

else if(strcmp(temp.substr(0,16).c_str(), "--grid-filename=") == 0) { 
    rest = temp.substr(16,strlen(temp.c_str())-16); 
    grid_filename.append(rest); //line 74! 
} 

Nun, wenn ich meinen Code ausführen, gibt valgrind mir diesen Fehler -

==5602== Address 0x45fdc30 is 0 bytes after a block of size 32 alloc'd 
==5602== at 0x402641D: operator new(unsigned int) (vg_replace_malloc.c:255) 
==5602== by 0x43039F7: std::string::_Rep::_S_create(unsigned int, unsigned int,  std::allocator<char> const&) (in /usr/lib/i386-linux-gnu/libstdc++.so.6.0.14) 
==5602== by 0x4304C77: std::string::_Rep::_M_clone(std::allocator<char> const&, unsigned int) (in /usr/lib/i386-linux-gnu/libstdc++.so.6.0.14) 
==5602== by 0x4304DA6: std::string::reserve(unsigned int) (in /usr/lib/i386-linux-gnu/libstdc++.so.6.0.14) 
==5602== by 0x43053E9: std::string::append(std::string const&) (in /usr/lib/i386-linux-gnu/libstdc++.so.6.0.14) 
==5602== by 0x804D5AE: get_command_line_args(int, char**) (main.cpp:74) 
==5602== by 0x804F138: main (main.cpp:244) 

ich drucken die beiden Adressen der Saiten und keiner von ihnen entsprechen derjenige valgrind sagt 0 Byte ist. Was fehlt mir hier?

Ich glaube, das führt zu meinem zweiten Fehler, weil ich Grid_filename an eine andere Funktion übergeben, die die Zeichenfolge über eine TCP-Verbindung sendet. Valgrind sagt mir

==5660== Syscall param socketcall.send(msg) points to unaddressable byte(s) 
==5660== at 0x404A9B1: send (socket.S:64) 
==5660== by 0x804F7C8: main (main.cpp:364) 

Kann mir jemand erklären, was das Problem ist? Jede Hilfe wäre willkommen. Ich kann bei Bedarf mehr über den Code liefern.

Answer 1

Über Ihren ersten Fehler: Wir hatten falsche Valves in Valgrind. Überprüfen Sie die documentation, um diese zu unterdrücken, vor allem, wenn sie nicht auf Ihren Code zeigen (und Sie haben überprüft, dass sie tatsächlich keine Probleme verursachen)

Answer 2

Hinweis: Dies ist keine Antwort, dies ist keine Code-Review-Website ... aber ich kann wirklich nicht darauf starren und weggehen.

Zunächst einige Funktionen, die wirklich in einer Toolbox helfen: zu

// Some free functions (because there are too many string types) 
inline char const* c_str(std::string const& s) { return s.c_str(); } 
inline size_t size(std::string const& s) { return s.size(); } 

inline char const* c_str(char const* s) { return s; } 
inline size_t size(char const* s) { return std::strlen(s); } 

template <size_t N> 
char const* c_str(char const (&s)[N]) { return s; } 
template <size_t N> 
size_t size(char const (&s)[N]) { return N - 1; } 

// A helper function (lowest common denominator) 
inline bool beginsWith(char const* big, size_t const bigSize, 
         char const* small, size_t const smallSize) 
{ 
    if (bigSize < smallSize) { return false; } 
    return std::memcmp(big, small, smallSize) == 0; 
} 

// The actual function, doing the adaptation from the various forms of string 
template <typename T, typename U> 
bool beginsWith(T& big, U& small) { 
    return beginsWith(c_str(big), size(big), c_str(small), size(small)); 
} 

// same with endsWith 

Und dann können Sie den Code recht effizient (ohne zusätzliche Speicherzuweisung) und mit mehr Lesbarkeit umschreiben:

static std::string const GridFilenameOpt = "--grid-filename="; 

// ... 
else if (beginsWith(temp, GridFilenameOpt)) { 
    grid_filename.append(temp, GridFilenameOpt.size(), std::string::npos); 
} 

Was für die eigentliche Frage nicht viel hilft, da Sie nicht den Code zeigen, der den Fehler verursacht.