multi: nilerr fixes

tapdb/assets_store.go

@@ -1153,7 +1153,8 @@ func (a *AssetStore) FetchAssetProofs(ctx context.Context,
 		// TODO(roasbeef): can modify the query to use IN somewhere
 		// instead? then would take input params and insert into
 		// virtual rows to use
-		for _, locator := range targetAssets {
+		for ind := range targetAssets {
+			locator := targetAssets[ind]
 			args, err := locatorToProofQuery(locator)
 			if err != nil {
 				return err

tapgarden/mock.go

@@ -5,7 +5,6 @@ import (
 	"context"
 	"encoding/hex"
 	"fmt"
-	"github.com/lightninglabs/taproot-assets/tapsend"
 	"math/rand"
 	"testing"
 	"time"
@@ -23,6 +22,7 @@ import (
 	"github.com/lightninglabs/taproot-assets/asset"
 	"github.com/lightninglabs/taproot-assets/internal/test"
 	"github.com/lightninglabs/taproot-assets/proof"
+	"github.com/lightninglabs/taproot-assets/tapsend"
 	"github.com/lightningnetwork/lnd/chainntnfs"
 	"github.com/lightningnetwork/lnd/keychain"
 	"github.com/lightningnetwork/lnd/lnwallet"
@@ -520,7 +520,7 @@ func (m *MockKeyRing) DeriveNextKey(ctx context.Context,
 
 	priv, err := btcec.NewPrivateKey()
 	if err != nil {
-		return keychain.KeyDescriptor{}, nil
+		return keychain.KeyDescriptor{}, err
 	}
 
 	loc := keychain.KeyLocator{

universe/auto_syncer.go

@@ -848,6 +848,7 @@ func (f *FederationEnvoy) SyncAssetInfo(ctx context.Context,
 		if err != nil {
 			log.Debugf("asset lookup for %v failed with remote"+
 				"server: %v", assetID.String(), addr.HostStr())
+			//lint:ignore nilerr failure is expected and logged
 			return nil
 		}