Fix memory leaks

Ooooscar · Aug 1, 2016 · bc8c004 · bc8c004
1 parent 8e51eb5
commit bc8c004
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 10 deletions.
diff --git a/AirPlay/reverseHTTP.cpp b/AirPlay/reverseHTTP.cpp
@@ -27,7 +27,7 @@ PTTH::PTTH()
 
 bool PTTH::conn(const char *address)
 {
-    char *addr_chr = (char *)malloc(strlen(address)+1);
+    char addr_chr[strlen(address) + 1];
     strcpy(addr_chr,address);
 
     char *addr_hostname = strtok(addr_chr, ":");
@@ -67,7 +67,6 @@ bool PTTH::conn(const char *address)
     }
 
     printf("[PTTH] %s Connected\n",addr_port);
-    free(addr_chr);
     return true;
 }
 

diff --git a/bilibili/Info.plist b/bilibili/Info.plist
@@ -233,7 +233,7 @@
 		</dict>
 	</array>
 	<key>CFBundleVersion</key>
-	<string>3083</string>
+	<string>3088</string>
 	<key>Fabric</key>
 	<dict>
 		<key>APIKey</key>

diff --git a/bilibili/Player/PlayerView.mm b/bilibili/Player/PlayerView.mm
@@ -386,17 +386,17 @@ - (NSDictionary *) getVideoInfo:(NSString *)url{
         }
     }
 
-    MediaInfoDLL::MediaInfo MI = *new MediaInfoDLL::MediaInfo;
-    MI.Open([url cStringUsingEncoding:NSUTF8StringEncoding]);
-    MI.Option(__T("Inform"), __T("Video;%Width%"));
-    NSString *width = [NSString stringWithCString:MI.Inform().c_str() encoding:NSUTF8StringEncoding];
-    MI.Option(__T("Inform"), __T("Video;%Height%"));
-    NSString *height = [NSString stringWithCString:MI.Inform().c_str() encoding:NSUTF8StringEncoding];
+    MediaInfoDLL::MediaInfo *MI = new MediaInfoDLL::MediaInfo;
+    MI->Open([url cStringUsingEncoding:NSUTF8StringEncoding]);
+    MI->Option(__T("Inform"), __T("Video;%Width%"));
+    NSString *width = [NSString stringWithCString:MI->Inform().c_str() encoding:NSUTF8StringEncoding];
+    MI->Option(__T("Inform"), __T("Video;%Height%"));
+    NSString *height = [NSString stringWithCString:MI->Inform().c_str() encoding:NSUTF8StringEncoding];
     NSDictionary *info = @{
                            @"width": width,
                            @"height": height,
                            };
-    delete &MI;
+    delete MI;
     return info;
 }