Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#3091 closed Bug (fixed)

tr_ctorSetDownloadDir() should check for empty string

Reported by: KyleK Owned by: KyleK
Priority: Normal Milestone: 1.93
Component: libtransmission Version: 1.92
Severity: Normal Keywords:
Cc:

Description

This is a valid RPC message:

      {
         "arguments": {
             "url": "http://example.com/test.torrent,
             "download-dir": ""
         },
         "method": "torrent-add"
      }

but it will set the download dir to an empty string, which as a result will result in Transmission trying to save the downloading files in the current directory.

I suggest that the function also tests if the string is empty and in that case does not set a custom download folder.

Attachments (1)

download-dir.patch (401 bytes) - added by KyleK 8 years ago.

Download all attachments as: .zip

Change History (4)

Changed 8 years ago by KyleK

comment:1 Changed 8 years ago by charles

  • Component changed from Transmission to libtransmission
  • Keywords backport-1.9x added
  • Milestone changed from None Set to 2.00
  • Owner set to KyleK

comment:2 Changed 8 years ago by charles

  • Resolution set to fixed
  • Status changed from new to closed

Fixed in trunk for 2.00 by r10427

comment:3 Changed 8 years ago by charles

  • Keywords backport-1.9x removed
  • Milestone changed from 2.00 to 1.93
Note: See TracTickets for help on using tickets.