fix the fixme.

kvtools · Apr 27, 2020 · 896c0ef · 896c0ef
1 parent 0f5d334
commit 896c0ef
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/store/dynamodb/dynamodb.go b/store/dynamodb/dynamodb.go
@@ -415,8 +415,9 @@ func (ddb *DynamoDB) AtomicDelete(key string, previous *store.KVPair) (bool, err
 	}
 
 	expAttr := make(map[string]*dynamodb.AttributeValue)
-	// FIXME previous.LastIndex may lead to a nil pointer dereference
-	expAttr[":lastRevision"] = &dynamodb.AttributeValue{N: aws.String(strconv.FormatUint(previous.LastIndex, 10))}
+	if previous != nil {
+		expAttr[":lastRevision"] = &dynamodb.AttributeValue{N: aws.String(strconv.FormatUint(previous.LastIndex, 10))}
+	}
 
 	req := &dynamodb.DeleteItemInput{
 		TableName: aws.String(ddb.tableName),