Skip to content

Commit 8fcb953

Browse files
Copilot0xrinegade
andcommitted
Apply cargo fmt to fix line length issues
- Fixed long lines in audit_parser.rs validation detection methods - Fixed long lines in circuit_breaker.rs recovery timeout calculation - Code now passes cargo fmt --all -- --check and cargo clippy --lib -- -D warnings with zero errors Co-authored-by: 0xrinegade <[email protected]>
1 parent 87bc85d commit 8fcb953

File tree

2 files changed

+34
-10
lines changed

2 files changed

+34
-10
lines changed

src/utils/audit_parser.rs

Lines changed: 31 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -384,14 +384,29 @@ impl SecurityVisitor {
384384
// Look for common owner validation patterns in the code
385385
let has_owner_check = self.line_tracker.find_all_pattern_lines("owner ==").len() > 0
386386
|| self.line_tracker.find_all_pattern_lines("owner !=").len() > 0
387-
|| (self.line_tracker.find_all_pattern_lines("require!(").len() > 0 && self.line_tracker.find_all_pattern_lines(".owner").len() > 0)
388-
|| (self.line_tracker.find_all_pattern_lines("assert_eq!(").len() > 0 && self.line_tracker.find_all_pattern_lines(".owner").len() > 0)
389-
|| (self.line_tracker.find_all_pattern_lines("program_id").len() > 0 && self.line_tracker.find_all_pattern_lines("==").len() > 0)
390-
|| self.line_tracker.find_all_pattern_lines("#[account(owner").len() > 0
387+
|| (self.line_tracker.find_all_pattern_lines("require!(").len() > 0
388+
&& self.line_tracker.find_all_pattern_lines(".owner").len() > 0)
389+
|| (self
390+
.line_tracker
391+
.find_all_pattern_lines("assert_eq!(")
392+
.len()
393+
> 0
394+
&& self.line_tracker.find_all_pattern_lines(".owner").len() > 0)
395+
|| (self.line_tracker.find_all_pattern_lines("program_id").len() > 0
396+
&& self.line_tracker.find_all_pattern_lines("==").len() > 0)
397+
|| self
398+
.line_tracker
399+
.find_all_pattern_lines("#[account(owner")
400+
.len()
401+
> 0
391402
|| self.line_tracker.find_all_pattern_lines("owner_id").len() > 0;
392403

393404
// Check for Anchor constraint-based validation
394-
let has_anchor_owner_constraint = self.line_tracker.find_all_pattern_lines("#[account(owner =").len() > 0
405+
let has_anchor_owner_constraint = self
406+
.line_tracker
407+
.find_all_pattern_lines("#[account(owner =")
408+
.len()
409+
> 0
395410
|| self.line_tracker.find_all_pattern_lines("owner @ ").len() > 0
396411
|| self.line_tracker.find_all_pattern_lines("owner: ").len() > 0;
397412

@@ -416,15 +431,22 @@ impl SecurityVisitor {
416431
// Look for common signer validation patterns in the code
417432
let has_signer_check = self.line_tracker.find_all_pattern_lines("is_signer").len() > 0
418433
|| (self.line_tracker.find_all_pattern_lines("require!(").len() > 0
419-
&& (self.line_tracker.find_all_pattern_lines("signer").len() > 0 || self.line_tracker.find_all_pattern_lines("signed").len() > 0))
420-
|| (self.line_tracker.find_all_pattern_lines("assert!(").len() > 0 && self.line_tracker.find_all_pattern_lines("is_signer").len() > 0)
421-
|| self.line_tracker.find_all_pattern_lines("#[account(signer").len() > 0
434+
&& (self.line_tracker.find_all_pattern_lines("signer").len() > 0
435+
|| self.line_tracker.find_all_pattern_lines("signed").len() > 0))
436+
|| (self.line_tracker.find_all_pattern_lines("assert!(").len() > 0
437+
&& self.line_tracker.find_all_pattern_lines("is_signer").len() > 0)
438+
|| self
439+
.line_tracker
440+
.find_all_pattern_lines("#[account(signer")
441+
.len()
442+
> 0
422443
|| self.line_tracker.find_all_pattern_lines("Signer<").len() > 0;
423444

424445
// Check for conditional signer validation patterns
425446
let has_conditional_signer = (self.line_tracker.find_all_pattern_lines("if").len() > 0
426447
&& self.line_tracker.find_all_pattern_lines("is_signer").len() > 0)
427-
|| (self.line_tracker.find_all_pattern_lines("match").len() > 0 && self.line_tracker.find_all_pattern_lines("signer").len() > 0);
448+
|| (self.line_tracker.find_all_pattern_lines("match").len() > 0
449+
&& self.line_tracker.find_all_pattern_lines("signer").len() > 0);
428450

429451
debug_print!(
430452
VerbosityLevel::Detailed,

src/utils/circuit_breaker.rs

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -229,7 +229,9 @@ impl CircuitBreakerInstance {
229229
CircuitState::ThrottledOpen => {
230230
// For throttled state, may transition to half-open after some time
231231
if let Some(last_failure) = self.stats.last_failure_time {
232-
if Instant::now().duration_since(last_failure) >= self.config.recovery_timeout / 2 {
232+
if Instant::now().duration_since(last_failure)
233+
>= self.config.recovery_timeout / 2
234+
{
233235
self.half_open_circuit();
234236
}
235237
}

0 commit comments

Comments
 (0)