diff options
author | rowanbeentje <rowan@beent.je> | 2010-02-10 01:32:05 +0000 |
---|---|---|
committer | rowanbeentje <rowan@beent.je> | 2010-02-10 01:32:05 +0000 |
commit | 13805614e6ed2131827bfa6c668b50a1b30da1e5 (patch) | |
tree | 8142178495af47518a2d693950d8623eec2ff2b5 /Source/TableDump.m | |
parent | d0fb716108641d3a8c197ef12041bf46b14e7294 (diff) | |
download | sequelpro-13805614e6ed2131827bfa6c668b50a1b30da1e5.tar.gz sequelpro-13805614e6ed2131827bfa6c668b50a1b30da1e5.tar.bz2 sequelpro-13805614e6ed2131827bfa6c668b50a1b30da1e5.zip |
Fix a number of memory leaks, and over-releases, as both a result of manual inspection of leaks and Clang static analysis.
Diffstat (limited to 'Source/TableDump.m')
-rw-r--r-- | Source/TableDump.m | 10 |
1 files changed, 8 insertions, 2 deletions
diff --git a/Source/TableDump.m b/Source/TableDump.m index 04942462..2e055f94 100644 --- a/Source/TableDump.m +++ b/Source/TableDump.m @@ -1746,7 +1746,10 @@ NSString *procedureName = [NSString stringWithFormat:@"%@", [proceduresList objectForKey:@"Name"]]; // Only proceed if the item was selected for export - if (![selectedItems containsObject:procedureName]) continue; + if (![selectedItems containsObject:procedureName]) { + [proceduresList release]; + continue; + } // Add the "drop" command if specified in the export dialog if ([addDropTableSwitch state] == NSOnState) { @@ -1756,7 +1759,10 @@ } // Only continue if the "create syntax" is specified in the export dialog - if ([addCreateTableSwitch state] == NSOffState) continue; + if ([addCreateTableSwitch state] == NSOffState) { + [proceduresList release]; + continue; + } //Definer is user@host but we need to escape it to `user`@`host` NSArray *procedureDefiner = [[proceduresList objectForKey:@"Definer"] componentsSeparatedByString:@"@"]; |