Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#5447 closed Bug (invalid)

Preference defaults get loaded from memory on OS X

Reported by: x190 Owned by: jordan
Priority: Normal Milestone: None Set
Component: Mac Client Version: 2.81+
Severity: Normal Keywords:
Cc:

Description

If there is no existing .plist with specific values for the various TR_KEYs and a previous version of Transmission was running, its values for the keys will be loaded from memory, rather than the ones in tr_sessionGetDefaultSettings(). Normally, one would not notice, but recently some default values have changed.

Transmission should be forced to use its own tr_sessionGetDefaultSettings() default values, if it does not find the key:value in the .plist, not just pull it from memory.

Everything in quark.c's static const struct tr_key_struct my_static[]should be initialized to NULL?

Apparently, OS X does not fully randomize memory space allocation.

Tested v2.81 r14152 which loaded default values from previously running v2.77, rather than its own new ones.

SL 10.6.8

Change History (3)

comment:1 Changed 8 years ago by x190

SL 10.6.8 + Mac Client

Summary quote:

loaded default values from previously running v2.77, rather than its own new ones.

I need to correct that. Both versions (v2.81+ and v2.77) must have been loading whatever value was in that memory space for that key. Here's some specific examples with a clean install and no user settings---no torrents, no resume files---all s/b defaults.

  • Up/Down? speed: defaults from session.c: 100/100. What I see in Mac Client preference window: 100 down:50 up.
  • Turtle (alt) speed: 50/50. What I see in Mac Client preference window: 10 down:10 up.
  • TR_KEY_download_queue_size,5: What I see in Mac Client preference window: 3.
  • TR_KEY_seed_queue_size, 10: What I see in Mac Client preference window: 3.

So, as stated in summary:

Transmission should be forced to use its own tr_sessionGetDefaultSettings() default values, if it does not find the key:value in the .plist, not just pull it from memory.

Everything in quark.c's static const struct tr_key_struct my_static[]should be initialized to NULL?

Last edited 8 years ago by x190 (previous) (diff)

comment:2 Changed 8 years ago by x190

  • Component changed from libtransmission to Mac Client
  • Resolution set to invalid
  • Status changed from new to closed

comment:3 Changed 8 years ago by x190

Frickasayin' macosx/Defaults.plist! :(

Note: See TracTickets for help on using tickets.