small fixes

a typo, using ongoing tx instead of making a new one, don't pass literal string
to formatting function.

found while working on quota support.

imapserver/server.go

@@ -1150,7 +1150,7 @@ func xcheckmailboxname(name string, allowInbox bool) string {
 	if isinbox {
 		xuserErrorf("special mailboxname Inbox not allowed")
 	} else if err != nil {
-		xusercodeErrorf("CANNOT", err.Error())
+		xusercodeErrorf("CANNOT", "%s", err)
 	}
 	return name
 }

webadmin/admin.go

@@ -1209,7 +1209,7 @@ Ensure a DNS TXT record like the following exists:
 		addf(&result.Instructions, instr)
 	}
 
-	// Hots TLSRPT
+	// Host TLSRPT
 	wg.Add(1)
 	var hostTLSRPTAddr smtp.Address
 	if mox.Conf.Static.HostTLSRPT.Localpart != "" {

webmail/api.go

@@ -628,7 +628,7 @@ func (w Webmail) MessageSubmit(ctx context.Context, m SubmitMessage) {
 				}
 			}
 
-			sentmb, err := bstore.QueryDB[store.Mailbox](ctx, acc.DB).FilterEqual("Sent", true).Get()
+			sentmb, err := bstore.QueryTx[store.Mailbox](tx).FilterEqual("Sent", true).Get()
 			if err == bstore.ErrAbsent {
 				// There is no mailbox designated as Sent mailbox, so we're done.
 				return