Skip to content

Commit a279db7

Browse files
author
HD Moore
committed
Check for sock.get / udp_sock.get issues
1 parent e806222 commit a279db7

File tree

1 file changed

+14
-0
lines changed

1 file changed

+14
-0
lines changed

tools/msftidy.rb

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -529,6 +529,18 @@ def check_newline_eof
529529
end
530530
end
531531

532+
def check_sock_get
533+
if @source =~ /\s+sock\.get(\s*|\(|\d+\s*|\d+\s*,\d+\s*)/m && @source !~ /sock\.get_once/
534+
info('Please use sock.get_once instead of sock.get')
535+
end
536+
end
537+
538+
def check_udp_sock_get
539+
if @source =~ /udp_sock\.get[^\d]+/m
540+
info('Please specify a timeout to udp_sock.get')
541+
end
542+
end
543+
532544
private
533545

534546
def load_file(file)
@@ -574,6 +586,8 @@ def run_checks(full_filepath)
574586
tidy.check_vuln_codes
575587
tidy.check_vars_get
576588
tidy.check_newline_eof
589+
tidy.check_sock_get
590+
tidy.check_udp_sock_get
577591
return tidy
578592
end
579593

0 commit comments

Comments
 (0)