your software is too fuzzy

  Рет қаралды 83,595

Low Level

Low Level

Күн бұрын

Пікірлер: 135
@Dev-Siri
@Dev-Siri Жыл бұрын
I already yell around 5-10 times a day at my computer
@piotrkaminski3443
@piotrkaminski3443 Жыл бұрын
I love this type of videos where you show a useful tool and an example using this tool, and what's even cooler is the fact that using it you were able to detect a bug that wasn't intentional
@mu11668B
@mu11668B Жыл бұрын
Just for fun though, there's a footgun hidden in the example code, too. As the recv buffer has a hardcoded length limit of 1024 bytes, directly casting the input buffer into a struct that contains a user-controlled length field is not really a good idea. If somehow the codebase got updated in a certain way and the memcpy destination was a heap allocation, it may lead to information leak. E.g. ask the server to echo a 65535-byte data chunk from a 1024-byte input.
@anon_y_mousse
@anon_y_mousse Жыл бұрын
Seems like nearly every video I'm warning about magic numbers. He really needs to tighten up his examples.
@Adam_Lyskawa
@Adam_Lyskawa Жыл бұрын
Well, the quicker way to ensure no crash here is just sanitize the input data. Then probably add a unit test for some edge cases. However - with much more complex example maybe using a fuzzer would be simpler, IDK. But THIS is probably the simplest explanation on how to use a fuzzer to begin with.
@macchiato_1881
@macchiato_1881 Жыл бұрын
Its always these hardcoded buffers that blow up in your face.
@MrAsddasdasda
@MrAsddasdasda Жыл бұрын
@@anon_y_mousse It's on purpose to get engagement from these comments.
@anon_y_mousse
@anon_y_mousse Жыл бұрын
@@MrAsddasdasda You may be right because I leave a comment every time just to say something about it.
@millax-ev6yz
@millax-ev6yz Жыл бұрын
Why is fuzzing better than boundary tests?...after watching I withdraw my question.
@adissentingopinion848
@adissentingopinion848 Жыл бұрын
I'm on the HDL/Hardware side where something like this is called Constrained Random Verification. Of course we do checks on boundary conditions in directed specific tests, but these devices have 30+ interfaces, so complex interactions can occur. Boundaries cover the 3 cases of too low, too high, or just right data inputs. But what if, say, if condition A AND Condition B And Condition C occur within x milliseconds to error out? A, B, and C are all within bounds, but this specific combination is deadly. For example, if on a server client 1 is somehow allowed to delete files in use by client 2 via an unsafe delete(file f) function, unless you know exactly how this exploit works you won't make a test for it. Two fake clients banging on a virtual keyboard, however, might find the right inputs over time to crash.
@millax-ev6yz
@millax-ev6yz Жыл бұрын
@@adissentingopinion848 I'm sold. I commented too early and using the Int vs unit is what did it, something that could be missed with the range. I have not used a tool like this so is it full path coverage or random? I wonder if the expense for full path doesn't become too high in terms of time....
@adissentingopinion848
@adissentingopinion848 Жыл бұрын
​@@millax-ev6yzIt's probably not going to get full code coverage UNLESS you explicitly get into a specific state for operation first. That "harness" mentioned for interfacing with the code can be very large and very customized. Simulations for hardware are terribly slow, but purely software testing ought to be rather fast up to a point. In hardware at least, you can set assertions that cover functional requirements such as message format. That way you don't have to error out, just capture the incorrect functionality from the harness itself.
@trexake
@trexake Жыл бұрын
Use -fsanitize=fuzzer,address and you should be able to find another bug in the parse code. If the input is less than the size of the struct you would read outside the memory. Does not always cause crash without address sanitizer. However not a bug in the program due to the receiving buffer size.
@matthias916
@matthias916 Жыл бұрын
"like literally yelling at the code" proceeds not to yell at the code
@mk72v2oq
@mk72v2oq Жыл бұрын
That's why I used to use unsigned everywhere by default, until negative values are explicitly required by design. And yes, using e.g. -1 magic value to represent things like a non-existent index is a bad design. Don't do it.
@joaquinnapan3237
@joaquinnapan3237 Жыл бұрын
what could I do instead for non-existent index??
@gigachad8810
@gigachad8810 Жыл бұрын
@@joaquinnapan3237 In rust you would do Option don't know about other languages.
@jacquesfaba55
@jacquesfaba55 Жыл бұрын
Error-as-types. Like Rust
@mk72v2oq
@mk72v2oq Жыл бұрын
@@joaquinnapan3237 Rust → Option C++ → std::optional C# → Nullable ... For languages with no option-like concept out of the box, you certainly can come up with something. E.g. in C you can utilize out parameter for the actual value and return the error code, or vice versa. Or return something like struct optional_uint32 { bool has_value; uint32_t value; }
@yeet1337
@yeet1337 Жыл бұрын
​@@gigachad8810in C?
@uuu12343
@uuu12343 Жыл бұрын
Satisfied customer here, been doing this for the last 10 years 10/10 - my code has feared me ever since
@LunarSoul255
@LunarSoul255 Жыл бұрын
Ah, there's a name for it. I do this regularly the manual way in my own projects, though granted those are all smaller projects where my scope of potential issues is "is there some way a user can force invalid data down this thing's throat". Useful to know if I ever manage to get a real job, lol(being a dev without a college degree is the dark souls of job hunting, I swear)
@wecann.clinic
@wecann.clinic 8 ай бұрын
Amazing brother, you have the gift of communicate complex concepts into simple terms. Thanks! Glad to find your channel! ;)
@dsdy1205
@dsdy1205 Жыл бұрын
I already do this every day
@holmybeer
@holmybeer Жыл бұрын
It would be really funny if he said "there's no more bugs in this code" and libfuzzer just crashed.
@mp_rho
@mp_rho Жыл бұрын
why did i think we might actually be yelling at code?
@sunofabeach9424
@sunofabeach9424 Жыл бұрын
the most reasonable action in the world of C programming
@coolbrotherf127
@coolbrotherf127 Жыл бұрын
I yell at my code, but it doesn't usually fix any bugs lol
@Kim_YoJong
@Kim_YoJong 11 ай бұрын
Because you're a fan of slamming desks.
@GeoffryGifari
@GeoffryGifari Жыл бұрын
at first we code safely by yelling in time elaborate rituals involving chanting, holy oils and incense is necessary to please the machine spirit and banish demonic bugs
@owenheckmann6962
@owenheckmann6962 Жыл бұрын
“Port 1337” that took me a second. Very funny
@markojojic6223
@markojojic6223 Жыл бұрын
Well, because I am so good at messing up function calls by using function pointers and structs/unions, I need no help. The code would yell either way nevertheless.
@markojojic6223
@markojojic6223 Жыл бұрын
Also, I am obsessed with keeping the memory usage low, so it's likely that I am gonna use a goofy assembly or stuff for my personal performance-intensive stuff. Especially on microcontrollers, but those don't count.
@Rose-ec6he
@Rose-ec6he Жыл бұрын
Segmentation fault (Core dumped)
@romsthe
@romsthe Жыл бұрын
2:38 in, I expect your issue is that you didn't check the length argument in your payload. This should pop up with many static analyzers. But I get it, it's just an example. Fuzzing is more for discovering weird edge cases and undefined behaviors as I understand it. Or I'm totally wrong and length was not the issue :D
@torarinvik4920
@torarinvik4920 Жыл бұрын
Also related but not the same: Property-based testing, those who haven't tried it will be amazed at it's usefulness.
@darkrasen
@darkrasen Жыл бұрын
I didn't quite catch why 7:45 is an issue. Would anyone mind please clarifying?
@Ferrolune
@Ferrolune Жыл бұрын
overflow probably, would be my first guess.
@turun_ambartanen
@turun_ambartanen Жыл бұрын
It checks if len>64, to prevent writing more than the allocated buffer. But negative numbers are also smaller than 64, so they also pass the check. The program then crashes in the memcp again, because it tries to copy a negative number of bytes.
@darkrasen
@darkrasen Жыл бұрын
@@turun_ambartanen thanks so much!
@louispetrick
@louispetrick Жыл бұрын
For an signed number we're using the two's complement to represent negative and positive numbers. Here the MSB decides whether the number is interpreted as an positive or negative number, where 0 = positive and 1 = negative. Looking at 7:45 for example, a hex value of 0xFF is represented in binary with 0b1111_1111. When assigned to a signed variable, this is actually a -1 in decimal. Since we use this variable "len" to access entries in an array, this will result in an error as it doesn't have negative entries to point at.
@darkrasen
@darkrasen Жыл бұрын
@@louispetrick thanks so much for the thorough explanation 😁
@abdulfatahmohammed6800
@abdulfatahmohammed6800 Жыл бұрын
Interesting, I have no idea this type of testing exists. Thanks man
@fulconandroadcone9488
@fulconandroadcone9488 Жыл бұрын
I was hoping for Torvalds kind of screaming at someone else code, but I guess this is fine.
@TechnopolisDotTV
@TechnopolisDotTV Жыл бұрын
As always chef's kiss!
@avishjha4030
@avishjha4030 Жыл бұрын
This is so cool, does something like this also exist in the Java world?
@trexake
@trexake Жыл бұрын
Jazzer does exactly that and is based on this.
@romanferguson4032
@romanferguson4032 Жыл бұрын
id love a video of you describing your linux setup. i use wsl and customize very few things but would love more insight into your setup for vim and tmux/whatever multi shell youre using
@huseyinemreeken3024
@huseyinemreeken3024 2 ай бұрын
oh it's basically baptising your code with fire
@abraarsameer9521
@abraarsameer9521 Жыл бұрын
Instructions unclear I’ve been yelling at code this whole time
@defnlife1683
@defnlife1683 Жыл бұрын
This was awesome. Fuzz all the things.
@PeepoStrong
@PeepoStrong Жыл бұрын
It reminds me some OOM error bug that got in project that was caused by using msgpack library (Java). The msgpack library deserializes byte stream into some objects - it was deserializing a base64 string to object. Apparently the library supports read a big array of bytes. Msgpack reads the message in sequence - does not know what data comes next - when the byte with flag for huge byte arrays comes in it pre-allocates array of 2^32-1 byte-elements. Found it because we had a malformed string that was not object we wanted to deserialize but rather random string. Later to confirm to architects that any idiot with msgpack documentation, paper and pencil can do it - prepared a base64 string on paper that mimic the good object to deserialize and then put the bytes of memory doom. They wanted to do some happy checking of first few bytes - after short demonstration - they changed their minds. With some java like fuzzer I would do that automatically (and probably the error could be found earlier), but fun of playing with bytes was awesome.
@sudo-gera
@sudo-gera Жыл бұрын
Flag '-g' makes stack traces of gdb or any sanitizer look pretty. Use it.
@StuartLoria
@StuartLoria Жыл бұрын
Those if statements are not very readable, but that is the prefered way, implementations details rather than intensions or requirements, if that is what people do then there is no alternative. Para pensar, señores.
@mytechnotalent
@mytechnotalent Жыл бұрын
I yell at code all day.
@PieroUlloa
@PieroUlloa Жыл бұрын
This seems great. I expect those eagle eyed developers saw the h- >len value, and thought to themselves about how user input is always evil :p but hey, the unsigned one did surprise me too! Luckily i like writing u32 u64 et al.
@bowiemtl
@bowiemtl Жыл бұрын
While this tool is awesome as is, is there any way to get it into an IDE? I think productivity would go up a lot of you can just select a function and some extension can do all the work for you returning only the result. Maybe I'm overlooking something that'd make you not want to use an extension like that but I think it'd be cool
@Ryan-jm3kw
@Ryan-jm3kw 4 ай бұрын
I feel like everyone should pen test their code with many other techniques also
@maxmyzer9172
@maxmyzer9172 Жыл бұрын
0:05 should have been the end lmao
@theblankuser
@theblankuser 10 ай бұрын
This is cool af
@lollertoaster
@lollertoaster Жыл бұрын
My favorite part of testing is "cat /dev/urandom | ./a.out" But that's specifically for testing proper error handling.
@ramsey2155
@ramsey2155 Жыл бұрын
This is how that belt makes your child stronger
@jefersonlemos4135
@jefersonlemos4135 Жыл бұрын
I thought you were doing like me and really cursing while programming, well that will prevent me from cursing
@itsjustrobby
@itsjustrobby 11 ай бұрын
What’s that you say? I’m not retarded I’m just left handed. This video just made me literally cry 😭
@pierreabbat6157
@pierreabbat6157 Жыл бұрын
I'm partial to American Fuzzy Lop, which compiles C++ code so that it knows which branches were taken. Can Rust code be fuzzed the same way, and is there a way to fuzz Haskell code that does something similar?
@TRex-fu7bt
@TRex-fu7bt Жыл бұрын
Does it statically analyze the wrapped function to deduce how to do the fuzzing? I’m struck by how it got the magic word immediately.
@FadkinsDiet
@FadkinsDiet Жыл бұрын
6:41 shell users everywhere are screaming at you there's no need to use cat, just use the
@versacebroccoli7238
@versacebroccoli7238 Жыл бұрын
Fuzzing is how Zenbleed was found!
@bryan0x05
@bryan0x05 Жыл бұрын
I really like the terminal environment you're using, how can I get my setup to look like that?
@funkdefied1
@funkdefied1 Жыл бұрын
Vim, prolly
@funkdefied1
@funkdefied1 Жыл бұрын
Neovim *
@gustavoaguilar7999
@gustavoaguilar7999 Жыл бұрын
i3wm
@1oglop1
@1oglop1 Жыл бұрын
Hi, sorry of the OT but I have a Rust/C question nobody was able to point me in the right direction. With redhook (unmaintained lib) I used LD_PRELOAD to override getenv which worked fine in NodeJs but Rust did not care about it at all. Do you know what is different or what should I read to understand how this all work? Thank you so much
@uis246
@uis246 Жыл бұрын
1:09 I already can guess r will be less than REQ_SIZE because recv doesn't have WAIT_ALL flag.
@FadkinsDiet
@FadkinsDiet Жыл бұрын
Even with WAIT_ALL maliciously crafted input could cause errors or DoS
@TheInspctrcat
@TheInspctrcat Жыл бұрын
Wow, sharp transitions should be smoothed out, otherwise this is an ultra-useful video
@EmilMacko
@EmilMacko Жыл бұрын
This guy: "Make your code safer by yelling at it. That's right, LITERALLY yelling at your code, in a very literal sense, can make your code, literally, safer. Legit stretch those vocal chords, open your mouth all the way, and just let out the biggest scream at the very top of your lungs, at your code, to make it safer!" This guy 20 seconds later: "So this process involves feeding random data into your program and..."
@sunofabeach9424
@sunofabeach9424 Жыл бұрын
clickbate my beloved
@AndreDeLimburger
@AndreDeLimburger Жыл бұрын
Even faster with a switch statement? You are already using a switch statement!
@lefteriseleftheriades7381
@lefteriseleftheriades7381 11 ай бұрын
At 2:33 i see the bug. He copies data based on the user inputed length on a buffer that ia limited to 64 bytes. I will watch more to see if this is what the fuzzer finds
@CjqNslXUcM
@CjqNslXUcM Жыл бұрын
simple good video
@drdca8263
@drdca8263 Жыл бұрын
How does this compare to concolic testing?
@davidpatry4195
@davidpatry4195 Жыл бұрын
pretty cool.
@backupmemories897
@backupmemories897 Жыл бұрын
how do i remove the path stuff inside my exe.. i see it exposes my directory in the exe.
@21centuryschizoid
@21centuryschizoid 10 ай бұрын
can you share the code with the bug ? thanks
@maxrepin6491
@maxrepin6491 Жыл бұрын
Although slightly off-topic, I was wondering if you could make a video explaining how cheat codes function in games like GTA San Andreas or Vice City. How they interact with the memory and what processes occur behind the scenes. I'd really appreciate a deep dive into this. Thank you!
@Hellscaped
@Hellscaped Жыл бұрын
They're just series of inputs that the game checks for and does something in response. Its not really complicated.
@EdKolis
@EdKolis Жыл бұрын
Yeah, you're probably thinking of Game Genie. Which I would like to see a video about how it works!
@elzabethtatcher9570
@elzabethtatcher9570 Жыл бұрын
I presume this fuzzer actually looks at the soruce code of the program, to predict how to best gain different outputs? It is not just random text generator?
@СергейМакеев-ж2н
@СергейМакеев-ж2н Жыл бұрын
It doesn't exactly look at the source code. Instead, it memorizes which random inputs caused which if-branches to be taken, and randomly mutates those inputs to "cover" as many routes through the program as possible. They call it "coverage-guided fuzzing".
@woosix7735
@woosix7735 Жыл бұрын
@@СергейМакеев-ж2н thanks for the explanation, it's pretty cool
@AvalancheGameArt
@AvalancheGameArt Жыл бұрын
I could see the bug even before the first test iteration...
@miniflint2423
@miniflint2423 Жыл бұрын
Hi ! I don’t understand the unsigned problem. Could someone explain?
@coolbrotherf127
@coolbrotherf127 Жыл бұрын
Signed numbers include negative numbers which the program had no way to handle so they caused a crash. By making it unsigned, it forces all values to be positive integer values 0-255 which the program could easily check.
@maxmuster7003
@maxmuster7003 Жыл бұрын
Limit the stack size to zero.😂
6 ай бұрын
I'll use Zig
@tomtravis858
@tomtravis858 Жыл бұрын
I love Rust
@raulr994
@raulr994 Жыл бұрын
*Pauses video 29 seconds in* You can't say LITERALLY yelling at your code if you don't mean to actually YELL at it vocally. That's the opposite of what LITERALLY means. :/
@amankashyap7842
@amankashyap7842 Жыл бұрын
i = 4; cout
@sudo-gera
@sudo-gera Жыл бұрын
C++ and C languages have very interesting thing: "Undefined behavior". This doesn't mean that behavior would be randomly chosen from "a set of possible behaviors". This means that behavior would be completely undefined. It can run into segfault or start erasing data on your PC. Anything is possible. Nothing is guaranteed. And for this case: In "a+b" expression, computation of a and b is not sequenced. They can happen in any order. Side effects of unsequenced operations cause Undefined behavior. Once it happened - nothing is guaranteed.
@amankashyap7842
@amankashyap7842 Жыл бұрын
@@sudo-gera thanks
@marbens
@marbens 9 ай бұрын
8:27 Wrong. switch statements are not faster than switch statements.
@granitium
@granitium Жыл бұрын
You weren't yelling at the code wth
@labkome
@labkome Жыл бұрын
Rust is good, but confusing for me
@abanoubha
@abanoubha Жыл бұрын
go fuzz 🎉
@sritharan20
@sritharan20 Жыл бұрын
goat
@uis246
@uis246 Жыл бұрын
2:30 not validated user input
@maxmuster7003
@maxmuster7003 Жыл бұрын
You need a pump gun to fix your code.😂
@kayakMike1000
@kayakMike1000 Жыл бұрын
Rust is silly.
@TatharNuar
@TatharNuar Жыл бұрын
a
@deusvult4214
@deusvult4214 Жыл бұрын
at 0:24 you said tha it's about "literally yelling at your code", but i didnt hear any yelling, though. Literally yelling means moving your face muscles to produce loud noise, yet during all your vide you were very calm. Why did you lie about this technique?
@xntumrfo9ivrnwf
@xntumrfo9ivrnwf Жыл бұрын
This is too powerful... people should just stick to Python
@dhaneshabhipraya
@dhaneshabhipraya Жыл бұрын
23h ago
@FranLegon
@FranLegon Жыл бұрын
0:24 two incorrect uses of "literally" in less than half a minute. Congrats
@tanishkmahakalkar761
@tanishkmahakalkar761 Жыл бұрын
First..!!!!😁🤩😍💯💥✨💫🔥👍🏻👏🏻✊🏻🤜🏻🤛🏻🙌🏻🫶🏻🙏🏻👌🏻
@CoolProgramer123
@CoolProgramer123 Жыл бұрын
third
@RussoIncendiario
@RussoIncendiario Жыл бұрын
first
using numbers in your code is bad
14:33
Low Level
Рет қаралды 144 М.
The purest coding style, where bugs are near impossible
10:25
Coderized
Рет қаралды 1 МЛН
Motorbike Smashes Into Porsche! 😱
00:15
Caters Clips
Рет қаралды 23 МЛН
Каха и лужа  #непосредственнокаха
00:15
Why You Shouldn't Nest Your Code
8:30
CodeAesthetic
Рет қаралды 2,8 МЛН
how do hackers exploit buffers that are too small?
8:25
Low Level
Рет қаралды 202 М.
they found another backdoor.
15:26
Low Level
Рет қаралды 151 М.
2 Years Of Learning C | Prime Reacts
22:24
ThePrimeTime
Рет қаралды 308 М.
the truth about ChatGPT generated code
10:35
Low Level
Рет қаралды 231 М.
if you view this image, YOU GET HACKED.
8:40
Low Level
Рет қаралды 388 М.
why does inheritance suck?
8:05
Low Level
Рет қаралды 231 М.
Finding The .webp Vulnerability in 8s (Fuzzing with AFL++)
24:11
LiveOverflow
Рет қаралды 62 М.
What is the Smallest Possible .EXE?
17:04
Inkbox
Рет қаралды 485 М.