loading...

re: SQL, strcat(), and more segmentation fault VIEW POST

FULL DISCUSSION
 

Quick tip.

strcat(3) should generally be avoided, unless you know 100% everything will fit.

Instead, in this case it would be better to use snprintf(3), though you can still get string truncation, or if you have GNU extensions you can use asprintf(3), which is like sprintf(3) but dynamically allocates the buffer.

Using snprint(3), your block of strcat()'s and malloc() can be reduced to...

#include <stdio.h>
...
char dsn[BUF_SZ];

snprintf(dsn, sizeof(dsn), "DSN=%s;UID=%s;PWD=%s",
         argv[1], argv[2], argv[3]);
 

Freaking brilliant! Worked like a charm, easier to read and configure. That's why I'm a noob.

Thanks,

 
 

Thanks, I'll have to give that a try.

Code of Conduct Report abuse