fix market actor publish_storage_deals

[noot]

Summary of changes
Changes introduced in this pull request:

• fix market actor publish_storage_deals panic, see lotus: https://github.com/filecoin-project/specs-actors/blob/master/actors/builtin/market/market_actor.go#L198

Reference issue to close (if applicable)

Closes n/a

Other information and links

[elmattic]

Not tested but pretty sure you can chain withs like this:

let msm = state.mutator(store)
    .with_pending_proposals(Permission::ReadOnly)
    .with_escrow_table(Permission::ReadOnly)
    .with_locked_table(Permission::ReadOnly);

This would allow msm binding staying immutable.

[noot]

hm, i get an error when i try that:

error[E0716]: temporary value dropped while borrowed
   --> vm/actor/src/builtin/market/mod.rs:268:23
    |
268 |           let msm = state.mutator(store)
    |                         ^^^^^^^^^^^^^^^^^^^^ creates a temporary which is freed while still in use
...
271 |           .with_locked_table(Permission::ReadOnly);
    |                                                   - temporary value is freed at the end of this statement
...
319 |               let client_balance_ok = msm
    |  _____________________________________-
320 | |                 .balance_covered(client, total_client_lockup.get(&client).unwrap())
    | |___________________________________________________________________________________- borrow later used here
    |
    = note: consider using a `let` binding to create a longer lived value

[elmattic]

There are places like here where withs are chained but then a build is used. Not sure at all if we need this.

[noot]

oh nice, yeah updated!

[elmattic]

lgtm