albertZhangTJ / sqlancer-lancerfuzz

SQLancer, with grammar-based fuzzing capability
http://www.sqlancer.com/
MIT License
1 stars 0 forks source link

Refactor fuzzer code to pass context as parameter #8

Closed albertZhangTJ closed 10 months ago

albertZhangTJ commented 11 months ago

As we develop more and more features, we are using more and more static data structures to keep track of states (e.g. Identifier Symbol Table, Used Identifier List, etc.). The structure of the generated fuzzer code is getting more and more messy.

For future work we can probably encapsulate all these information in some kind of Context class and pass it as parameter. This shall help keep the fuzzer AST itself clean.

albertZhangTJ commented 10 months ago

Thought twice about this one, probably doesn't make that much a difference. Especially when we wrap all the context related stuff into a single function call. Will close for now.

albertZhangTJ commented 10 months ago

Wont fix