From: Matt Joiner <anacrolix@gmail.com>
Date: Tue, 20 Sep 2016 08:39:07 +0000 (+1000)
Subject: Fixes for changes to metainfo.MetaInfo.UnmarshalInfo
X-Git-Tag: v1.0.0~576
X-Git-Url: http://www.git.stargrave.org/?a=commitdiff_plain;h=594053609ce70e41461b08cbdbe21a0b93c58dfc;p=btrtrc.git

Fixes for changes to metainfo.MetaInfo.UnmarshalInfo
---

diff --git a/client_test.go b/client_test.go
index e23f7250..dbcd02f8 100644
--- a/client_test.go
+++ b/client_test.go
@@ -730,7 +730,8 @@ func testAddTorrentPriorPieceCompletion(t *testing.T, alreadyCompleted bool, csf
 	greetingDataTempDir, greetingMetainfo := testutil.GreetingTestTorrent()
 	defer os.RemoveAll(greetingDataTempDir)
 	filePieceStore := csf(fileCache)
-	info := greetingMetainfo.UnmarshalInfo()
+	info, err := greetingMetainfo.UnmarshalInfo()
+	require.NoError(t, err)
 	ih := greetingMetainfo.HashInfoBytes()
 	greetingData, err := storage.NewClient(filePieceStore).OpenTorrent(&info, ih)
 	require.NoError(t, err)
diff --git a/cmd/magnet-metainfo/main.go b/cmd/magnet-metainfo/main.go
index f7b7dce0..e231f171 100644
--- a/cmd/magnet-metainfo/main.go
+++ b/cmd/magnet-metainfo/main.go
@@ -29,7 +29,7 @@ func main() {
 			<-t.GotInfo()
 			mi := t.Metainfo()
 			t.Drop()
-			f, err := os.Create(mi.UnmarshalInfo().Name + ".torrent")
+			f, err := os.Create(t.Info().Name + ".torrent")
 			if err != nil {
 				log.Fatalf("error creating torrent metainfo file: %s", err)
 			}
diff --git a/cmd/torrent-magnet/main.go b/cmd/torrent-magnet/main.go
index 50e57b16..ee0ea2ec 100644
--- a/cmd/torrent-magnet/main.go
+++ b/cmd/torrent-magnet/main.go
@@ -17,7 +17,11 @@ func main() {
 		fmt.Fprintf(os.Stderr, "error reading metainfo from stdin: %s", err)
 		os.Exit(1)
 	}
-	info := mi.UnmarshalInfo()
+	info, err := mi.UnmarshalInfo()
+	if err != nil {
+		fmt.Fprintf(os.Stderr, "error unmarshalling info: %s", err)
+		os.Exit(1)
+	}
 
 	fmt.Fprintf(os.Stdout, "%s\n", mi.Magnet(info.Name, mi.HashInfoBytes()).String())
 }
diff --git a/cmd/torrent-metainfo-pprint/main.go b/cmd/torrent-metainfo-pprint/main.go
index 4f99a997..91e7c7aa 100644
--- a/cmd/torrent-metainfo-pprint/main.go
+++ b/cmd/torrent-metainfo-pprint/main.go
@@ -29,7 +29,11 @@ func main() {
 			log.Print(err)
 			continue
 		}
-		info := metainfo.UnmarshalInfo()
+		info, err := metainfo.UnmarshalInfo()
+		if err != nil {
+			log.Printf("error unmarshalling info: %s", err)
+			continue
+		}
 		if flags.JustName {
 			fmt.Printf("%s\n", info.Name)
 			continue
diff --git a/cmd/torrent-verify/main.go b/cmd/torrent-verify/main.go
index 7f329302..ba64d801 100644
--- a/cmd/torrent-verify/main.go
+++ b/cmd/torrent-verify/main.go
@@ -49,7 +49,10 @@ func main() {
 	if err != nil {
 		log.Fatal(err)
 	}
-	info := metaInfo.UnmarshalInfo()
+	info, err := metaInfo.UnmarshalInfo()
+	if err != nil {
+		log.Fatalf("error unmarshalling info: %s", err)
+	}
 	mMapSpan := &mmap_span.MMapSpan{}
 	if len(info.Files) > 0 {
 		for _, file := range info.Files {
diff --git a/issue97_test.go b/issue97_test.go
index 2695e5b7..5e1ab8f9 100644
--- a/issue97_test.go
+++ b/issue97_test.go
@@ -18,7 +18,8 @@ func TestHashPieceAfterStorageClosed(t *testing.T) {
 	cs := storage.NewClient(storage.NewFile(td))
 	tt := &Torrent{}
 	mi := testutil.GreetingMetaInfo()
-	info := mi.UnmarshalInfo()
+	info, err := mi.UnmarshalInfo()
+	require.NoError(t, err)
 	tt.info = &info
 	tt.makePieces()
 	tt.storage, err = cs.OpenTorrent(tt.info, mi.HashInfoBytes())