Skip to content

Commit dc557cb

Browse files
hczphnclaude
andcommitted
fix: restore verifier setup from shared memory to fix verify panic
The previous optimization skipped reading PCS setup from shared memory and returned empty defaults, which caused verify to panic on v_keys lookup (unwrap on None). Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent 2465d52 commit dc557cb

File tree

1 file changed

+5
-5
lines changed
  • expander_compiler/src/zkcuda/proving_system/expander_parallelized

1 file changed

+5
-5
lines changed

expander_compiler/src/zkcuda/proving_system/expander_parallelized/client_utils.rs

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -140,11 +140,11 @@ where
140140

141141
setup_timer.stop();
142142

143-
// Skip reading PCS setup from shared memory; return default to reduce memory
144-
(
145-
ExpanderProverSetup::default(),
146-
ExpanderVerifierSetup::default(),
147-
)
143+
// Prover setup not needed on client side (server does the proving).
144+
// Verifier setup is required for verification, so read it from shared memory.
145+
let (_prover_setup, verifier_setup) =
146+
SharedMemoryEngine::read_pcs_setup_from_shared_memory::<C::FieldConfig, C::PCSConfig>();
147+
(ExpanderProverSetup::default(), verifier_setup)
148148
}
149149

150150
pub fn client_send_witness_and_prove<C, ECCConfig>(

0 commit comments

Comments
 (0)