Add workers #2

Parean · 2017-12-05T01:02:18Z

Сырой код, я его ещё допишу

alexbers

почитал, норм, что-то принял

alexbers · 2017-12-05T09:29:14Z

README.md

 # Exploit Farm #

-The utility for CTF hacker competition for lauching sploits for all teams
+The utility for CTF hacker competition for launching sploits for all teams


Опечатка исправлена, спасибо

alexbers · 2017-12-05T09:29:56Z

README.md

 ## Prepare ##
 1. Set FLAG_FORMAT regexp in flag_format.py
-2. Set TEAMS in team_list.py
+2. Set teams in team_list.py


По PEP8 константы пишутся заглавными буквами

Да, знаю, но имеет смысл в течение игры обновлять список команд, чтобы взаимодействовать только с теми, которые сейчас активны. На Ructf_2017 мы это сделали на скорую руку с помощью парсинга страницы регулярным выражением, добавив функцию generate_teams, её ещё нужно будет привести в нормальный вид. Поэтому я и изменил teams, так как это больше не константа (хотя для тренировок можно будет просто комментировать вызовы этой функции, саму функцию, и заполнять teams руками, как и раньше)

в этом месте, наверно, можно написать отдельно генератор таких файлов, который может пытаться, например, законнектиться по TCP к каждой команде. Но вот как это сделать просто - я не придумал

alexbers · 2017-12-05T09:30:17Z

flag_format.py

@@ -1,2 +1,2 @@
 # put the regexp for flag here
-FLAG_FORMAT = b"[0-9A-Fa-f]{32}"
+FLAG_FORMAT = b"[Mm]oved"


В чём смысл этого изменения?

Это отладочые изменения для локального тестирования, я потом верну всё как было

alexbers · 2017-12-05T09:35:32Z

start_posting.py

 FLAGS_IN_SUMBIT_ITERATION = 100
-RESTART_DELAY             = 2       # in sec
 FLAGS_GLOB                = "flags/*.txt"
+HOST                      = '127.0.0.1'  # checksystem hostaddr here


Да, наверно так будет лучше, закоммитил

alexbers · 2017-12-05T09:36:59Z

start_posting.py

 def log(text):
-    print(strftime("%H:%M:%S") + " " + text)
-
+    print(strftime("%H:%M:%S ") + text)


Тут не вижу разницы, но нахожу исходный вариант более семантически правильным. Тоесть взять время добавить к нему пробел, а потом текст. В новом варианте берётся время в формате с пробелом в конце и добавляется текст

alexbers · 2017-12-05T10:28:57Z

start_sploit.py

+        lock.acquire()
+         # We need to create a new object so we can later find new flags
+         # comparing two different sets
        last_flags = [set(o.flags) for o in owners]


вроде в дефолтной реализации питона есть GIL, который решает проблему с неатомарностью set'ов

Не очень понял про атомарность. Мне было непонятно, зачем писать set(o.flags), вместо o.flags, если o.flags -- это уже set. Я думал это ошибка, и изначально o.flags был списком, а потом в конструкторе его сразу сделали множеством, а здесь и в 181 строчке просто забыли добавить правки. Я убрал set в обоих местах, но потом понял, что здесь его нужно оставить, потому что в дальнейшем нам нужно узнать какие флаги добавились, а сделать это можно только сравнив два разных множества, которые друг от друга не зависят. Поэтому я добавил комментарий, но похоже не очень понятный

alexbers · 2017-12-05T10:30:58Z

start_sploit.py

+            owners_queue.put(o)

-        curr_flags = [set(o.flags) for o in owners]
+        sleep(1)  # show stat at zero second


тут раньше был смысл, чтобы статистика писалась в 00 секунд каждой минуты. Т.е. не, например, в 12:09:11, 12:10:11, 12:11:11, а в 12:10:00, 12:11:00, 12:12:00 и т.д

Это отладочые изменения для локального тестирования, я потом верну всё как было

alexbers · 2017-12-05T10:33:18Z

start_sploit.py

+for i in range(multiprocessing.cpu_count()):
+    owners_queue.put(None)
+for w in workers:
+    w.join()


тут возникнет проблема с сплойтами, которые долго работают. Например пытаются содединиться с кем-то и ждут таймаута.

Я посмотрю ещё как правильно завершать работу воркеров

alexbers · 2017-12-05T10:33:56Z

team_list.py

+
+
+# teams = {}
+# def generate_teams():


код специфичен для конкретной странички конкретных соревнований

Да, его надо переписать

alexbers · 2017-12-05T10:34:20Z

team_list.py

-    "team1"  : "127.0.0.1",
-    "team2"  : "127.0.0.2",
+teams = {
+    "team1"  : "172.217.4.3",


хм:) интересно что это за адрес

alexbers · 2017-12-05T11:14:04Z

Сейчас проблема кода, на мой взгляд - в том что он создаёт по два потока и один процесс на команду. В линуксе от одного потока можно избаваться, если заменить чтение строки на select с таймаутом. Но в windows такое не работает :(

Add workers

fcdad61

alexbers reviewed Dec 5, 2017

View reviewed changes

Add workers #2

Are you sure you want to change the base?

Add workers #2

Uh oh!

Conversation

Parean commented Dec 5, 2017 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

alexbers left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

alexbers commented Dec 5, 2017

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Parean commented Dec 5, 2017 •

edited

Loading