diff options
| author | Xavier Roche <xroche@users.noreply.github.com> | 2014-05-02 15:13:29 +0000 |
|---|---|---|
| committer | Xavier Roche <xroche@users.noreply.github.com> | 2014-05-02 15:13:29 +0000 |
| commit | 5544f503bf0fcfd050b4e338e8ec7b22e2f53b9a (patch) | |
| tree | 2d65f1fd285c450cbb1c676697ad13b5a47a835c /src/htsname.c | |
| parent | b3fa8537c411e6e2d53044b1d5d20c361d2ad17d (diff) | |
Big cleanup in functions writing to a char buffer without proper size boundary.
Diffstat (limited to 'src/htsname.c')
| -rw-r--r-- | src/htsname.c | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/src/htsname.c b/src/htsname.c index 500e5a6..1a3ee47 100644 --- a/src/htsname.c +++ b/src/htsname.c @@ -328,8 +328,8 @@ int url_savename(char *adr_complete, char *fil_complete, char *save, is_html = ishtml(opt, fil); switch (is_html) { /* .html,.shtml,.. */ case 1: - if ((strfield2(get_ext(OPT_GET_BUFF(opt), fil), "html") == 0) - && (strfield2(get_ext(OPT_GET_BUFF(opt), fil), "htm") == 0) + if ((strfield2(get_ext(OPT_GET_BUFF(opt), OPT_GET_BUFF_SIZE(opt), fil), "html") == 0) + && (strfield2(get_ext(OPT_GET_BUFF(opt), OPT_GET_BUFF_SIZE(opt), fil), "htm") == 0) ) { strcpybuff(ext, "html"); ext_chg = 1; @@ -845,7 +845,7 @@ int url_savename(char *adr_complete, char *fil_complete, char *save, *d++ = *c++; } *d = '\0'; - d = unescape_http(catbuff, name[0]); + d = unescape_http(catbuff, sizeof(catbuff), name[0]); if (d && *d) { strcpybuff(b, d); /* value */ b += strlen(b); @@ -1714,7 +1714,8 @@ char *url_savename_refname_fullpath(httrackp * opt, const char *adr, char digest_filename[64]; url_savename_refname(adr, fil, digest_filename); - return fconcat(OPT_GET_BUFF(opt), StringBuff(opt->path_log), digest_filename); + return fconcat(OPT_GET_BUFF(opt), OPT_GET_BUFF_SIZE(opt), + StringBuff(opt->path_log), digest_filename); } /* remove refname if any */ |