Changeset 14634


Ignore:
Timestamp:
Dec 25, 2015, 11:34:35 AM (6 years ago)
Author:
mikedld
Message:

Ongoing refactoring (use size_t instead of int)

Location:
trunk
Files:
16 edited

Legend:

Unmodified
Added
Removed
  • trunk/libtransmission/announcer-http.c

    r14619 r14634  
    115115        evutil_inet_ntop (AF_INET6, ipv6, ipv6_readable, INET6_ADDRSTRLEN);
    116116        evbuffer_add_printf (buf, "&ipv6=");
    117         tr_http_escape (buf, ipv6_readable, -1, true);
     117        tr_http_escape (buf, ipv6_readable, TR_BAD_SIZE, true);
    118118    }
    119119
     
    216216                fprintf (stderr, "%s", "Announce response was not in benc format\n");
    217217            else {
    218                 int i, len;
     218                size_t i, len;
    219219                char * str = tr_variantToStr (&benc, TR_VARIANT_FMT_JSON, &len);
    220220                fprintf (stderr, "%s", "Announce response:\n< ");
     
    371371                fprintf (stderr, "%s", "Scrape response was not in benc format\n");
    372372            else {
    373                 int i, len;
     373                size_t i, len;
    374374                char * str = tr_variantToStr (&top, TR_VARIANT_FMT_JSON, &len);
    375375                fprintf (stderr, "%s", "Scrape response:\n< ");
  • trunk/libtransmission/magnet.c

    r14361 r14634  
    4242
    4343static void
    44 base32_to_sha1 (uint8_t * out, const char * in, const int inlen)
    45 {
    46   const int outlen = 20;
    47   int i, index, offset;
     44base32_to_sha1 (uint8_t * out, const char * in, const size_t inlen)
     45{
     46  const size_t outlen = 20;
     47  size_t i, index, offset;
    4848
    4949  memset (out, 0, 20);
     
    122122          const char * val = delim == NULL ? NULL : delim + 1;
    123123          const char * next = strchr (delim == NULL ? key : val, '&');
    124           int keylen, vallen;
     124          size_t keylen, vallen;
    125125
    126126          if (delim != NULL)
    127             keylen = delim - key;
     127            keylen = (size_t) (delim - key);
    128128          else if (next != NULL)
    129             keylen = next - key;
     129            keylen = (size_t) (next - key);
    130130          else
    131131            keylen = strlen (key);
     
    134134            vallen = 0;
    135135          else if (next != NULL)
    136             vallen = next - val;
     136            vallen = (size_t) (next - val);
    137137          else
    138138            vallen = strlen (val);
     
    141141            {
    142142              const char * hash = val + 9;
    143               const int hashlen = vallen - 9;
     143              const size_t hashlen = vallen - 9;
    144144
    145145              if (hashlen == 40)
  • trunk/libtransmission/metainfo.c

    r14355 r14634  
    413413                      tr_info           * inf,
    414414                      bool              * hasInfoDict,
    415                       int               * infoDictLength,
    416                       const tr_variant     * meta_in)
     415                      size_t            * infoDictLength,
     416                      const tr_variant  * meta_in)
    417417{
    418418  int64_t i;
     
    469469  else
    470470    {
    471       int len;
     471      size_t len;
    472472      char * bstr = tr_variantToStr (infoDict, TR_VARIANT_FMT_BENC, &len);
    473473      tr_sha1 (inf->hash, bstr, len, NULL);
     
    577577                  tr_info          * inf,
    578578                  bool             * hasInfoDict,
    579                   int              * infoDictLength)
     579                  size_t           * infoDictLength)
    580580{
    581581  const char * badTag = tr_metainfoParseImpl (session,
  • trunk/libtransmission/metainfo.h

    r14241 r14634  
    2222                        tr_info           * setmeInfo,
    2323                        bool              * setmeHasInfoDict,
    24                         int               * setmeInfoDictLength);
     24                        size_t            * setmeInfoDictLength);
    2525
    2626void tr_metainfoRemoveSaved (const tr_session * session,
  • trunk/libtransmission/peer-msgs.c

    r14619 r14634  
    19561956    {
    19571957        char * data;
    1958         int dataLen;
     1958        size_t dataLen;
    19591959        bool ok = false;
    19601960
  • trunk/libtransmission/torrent-ctor.c

    r14532 r14634  
    112112        err = -1;
    113113    else {
    114         int len;
     114        size_t len;
    115115        tr_variant tmp;
    116116        char * str;
  • trunk/libtransmission/torrent-magnet.c

    r14619 r14634  
    9999}
    100100
    101 static int
     101static size_t
    102102findInfoDictOffset (const tr_torrent * tor)
    103103{
    104104  size_t fileLen;
    105105  uint8_t * fileContents;
    106   int offset = 0;
     106  size_t offset = 0;
    107107
    108108  /* load the file, and find the info dict's offset inside the file */
     
    117117          if (tr_variantDictFindDict (&top, TR_KEY_info, &infoDict))
    118118            {
    119               int infoLen;
     119              size_t infoLen;
    120120              char * infoContents = tr_variantToStr (infoDict, TR_VARIANT_FMT_BENC, &infoLen);
    121121              const uint8_t * i = (const uint8_t*) tr_memmem ((char*)fileContents, fileLen, infoContents, infoLen);
     
    146146
    147147void*
    148 tr_torrentGetMetadataPiece (tr_torrent * tor, int piece, int * len)
     148tr_torrentGetMetadataPiece (tr_torrent * tor, int piece, size_t * len)
    149149{
    150150  char * ret = NULL;
     
    161161
    162162      assert (tor->infoDictLength > 0);
    163       assert (tor->infoDictOffset >= 0);
    164163
    165164      fd = tr_sys_file_open (tor->info.torrent, TR_SYS_FILE_READ, 0, NULL);
    166165      if (fd != TR_BAD_SYS_FILE)
    167166        {
    168           const int o = piece * METADATA_PIECE_SIZE;
     167          const size_t o = piece * METADATA_PIECE_SIZE;
    169168
    170169          if (tr_sys_file_seek (fd, tor->infoDictOffset + o, TR_SEEK_SET, NULL, NULL))
    171170            {
    172               const int l = o + METADATA_PIECE_SIZE <= tor->infoDictLength
    173                           ? METADATA_PIECE_SIZE
    174                           : tor->infoDictLength - o;
     171              const size_t l = o + METADATA_PIECE_SIZE <= tor->infoDictLength
     172                             ? METADATA_PIECE_SIZE
     173                             : tor->infoDictLength - o;
    175174
    176175              if (0<l && l<=METADATA_PIECE_SIZE)
     
    178177                  char * buf = tr_new (char, l);
    179178                  uint64_t n;
    180                   if (tr_sys_file_read (fd, buf, l, &n, NULL) && n == (unsigned int) l)
     179                  if (tr_sys_file_read (fd, buf, l, &n, NULL) && n == l)
    181180                    {
    182181                      *len = l;
     
    260259                  bool hasInfo;
    261260                  tr_info info;
    262                   int infoDictLength;
     261                  size_t infoDictLength;
    263262
    264263                  /* remove any old .torrent and .resume files */
     
    392391    {
    393392      evbuffer_add_printf (s, "%s", "&dn=");
    394       tr_http_escape (s, name, -1, true);
     393      tr_http_escape (s, name, TR_BAD_SIZE, true);
    395394    }
    396395
     
    398397    {
    399398      evbuffer_add_printf (s, "%s", "&tr=");
    400       tr_http_escape (s, inf->trackers[i].announce, -1, true);
     399      tr_http_escape (s, inf->trackers[i].announce, TR_BAD_SIZE, true);
    401400    }
    402401
     
    404403    {
    405404      evbuffer_add_printf (s, "%s", "&ws=");
    406       tr_http_escape (s, inf->webseeds[i], -1, true);
     405      tr_http_escape (s, inf->webseeds[i], TR_BAD_SIZE, true);
    407406    }
    408407
  • trunk/libtransmission/torrent-magnet.h

    r14241 r14634  
    2323};
    2424
    25 void* tr_torrentGetMetadataPiece (tr_torrent * tor, int piece, int * len);
     25void* tr_torrentGetMetadataPiece (tr_torrent * tor, int piece, size_t * len);
    2626
    2727void tr_torrentSetMetadataPiece (tr_torrent * tor, int piece, const void * data, int len);
  • trunk/libtransmission/torrent.c

    r14613 r14634  
    981981                  tr_info        * setmeInfo,
    982982                  bool           * setmeHasInfo,
    983                   int            * dictLength,
     983                  size_t         * dictLength,
    984984                  int            * setme_duplicate_id)
    985985{
     
    10401040tr_torrentNew (const tr_ctor * ctor, int * setme_error, int * setme_duplicate_id)
    10411041{
    1042   int len;
     1042  size_t len;
    10431043  bool hasInfo;
    10441044  tr_info tmpInfo;
  • trunk/libtransmission/torrent.h

    r14532 r14634  
    168168
    169169    /* Length, in bytes, of the "info" dict in the .torrent file. */
    170     int infoDictLength;
     170    size_t infoDictLength;
    171171
    172172    /* Offset, in bytes, of the beginning of the "info" dict in the .torrent file.
     
    174174     * Used by the torrent-magnet code for serving metainfo to peers.
    175175     * This field is lazy-generated and might not be initialized yet. */
    176     int infoDictOffset;
     176    size_t infoDictOffset;
    177177
    178178    /* Where the files are now.
  • trunk/libtransmission/variant-test.c

    r14613 r14634  
    160160  char * saved;
    161161  const size_t    len = strlen (str);
    162   int savedLen;
     162  size_t savedLen;
    163163  int err;
    164164
     
    196196  const char * end;
    197197  int err;
    198   int len;
     198  size_t len;
    199199  int64_t i;
    200200  char * saved;
     
    436436{
    437437  int i;
    438   int len;
     438  size_t len;
    439439  int err;
    440440  char * in;
     
    510510  double realVal;
    511511  bool boolVal;
    512   int len;
     512  size_t len;
    513513  char * benc;
    514514  const char * end;
  • trunk/libtransmission/variant.c

    r14633 r14634  
    12001200
    12011201char*
    1202 tr_variantToStr (const tr_variant * v, tr_variant_fmt fmt, int * len)
     1202tr_variantToStr (const tr_variant * v, tr_variant_fmt fmt, size_t * len)
    12031203{
    12041204  struct evbuffer * buf = tr_variantToBuf (v, fmt);
    1205   size_t n;
    1206   char * ret = evbuffer_free_to_str (buf, &n);
    1207   if (len != NULL)
    1208     *len = (int) n;
    1209   return ret;
     1205  return evbuffer_free_to_str (buf, len);
    12101206}
    12111207
  • trunk/libtransmission/variant.h

    r14633 r14634  
    117117char* tr_variantToStr (const tr_variant * variant,
    118118                       tr_variant_fmt     fmt,
    119                        int              * len);
     119                       size_t           * len);
    120120
    121121struct evbuffer * tr_variantToBuf (const tr_variant * variant,
  • trunk/libtransmission/web.c

    r14532 r14634  
    608608tr_http_escape (struct evbuffer  * out,
    609609                const char       * str,
    610                 int                len,
     610                size_t             len,
    611611                bool               escape_slashes)
    612612{
    613   const char * end;
    614 
    615   if ((len < 0) && (str != NULL))
     613  if (str == NULL)
     614    return;
     615
     616  if (len == TR_BAD_SIZE)
    616617    len = strlen (str);
    617618
    618   for (end=str+len; str && str!=end; ++str)
     619  for (const char * end = str + len; str != end; ++str)
    619620    {
    620621      if ((*str == ',') || (*str == '-')
     
    631632
    632633char *
    633 tr_http_unescape (const char * str, int len)
     634tr_http_unescape (const char * str, size_t len)
    634635{
    635636  char * tmp = curl_unescape (str, len);
  • trunk/libtransmission/web.h

    r14572 r14634  
    6969void tr_webGetTaskInfo (struct tr_web_task * task, tr_web_task_info info, void * dst);
    7070
    71 void tr_http_escape (struct evbuffer *out, const char *str, int len, bool escape_slashes);
     71void tr_http_escape (struct evbuffer *out, const char *str, size_t len, bool escape_slashes);
    7272
    7373void tr_http_escape_sha1 (char * out, const uint8_t * sha1_digest);
    7474
    75 char* tr_http_unescape (const char * str, int len);
     75char* tr_http_unescape (const char * str, size_t len);
    7676
    7777#ifdef __cplusplus
  • trunk/qt/RpcClient.cc

    r14552 r14634  
    143143        request.setRawHeader (TR_RPC_SESSION_ID_HEADER, mySessionId.toUtf8 ());
    144144
    145       int rawJsonDataLength;
     145      size_t rawJsonDataLength;
    146146      char * rawJsonData = tr_variantToStr (json.get (), TR_VARIANT_FMT_JSON_LEAN, &rawJsonDataLength);
    147147      QByteArray jsonData (rawJsonData, rawJsonDataLength);
Note: See TracChangeset for help on using the changeset viewer.