Unlimited string copying V512 A call of the 'sprintf' function will lead to overflow of the buffer 'fullpath'. Output = '\0' // > 16) != 20) || ((request > 16) != 9))Īgain, it looks like the problem of using the wrong operator - either || instead of & or = instead of != because the variable can't store the values 20 and 9 at the same time. Subprocess(char *const argv, str_handle_lines_t linehandler, void *data) No error handling V557 Array underrun is possible. But this defect isn't dangerous because all previously allocated memory will be freed by the operating system once the program terminates. The first error is found immediately in the main function: the code following the return statement was meant to free the memory allocated earlier. Return handle_disconnect_reason(deactivated, ext_disc_reason) Unreachable code V779 Unreachable code detected. Compare that with FreeRDP with its 320 KLOC. The project is pretty old, though: it was released for the first time on April 4, 2001, and is 17 years old, as of this writing.Īs I already said, the project is very small - about 30 KLOC, which is a bit strange considering its age. It is used as a default client on ReactOS, and you can also find third-party graphical front-ends to go with it. It can also run on Windows if built under Cygwin. rdesktop rdesktop is a free RDP client for UNIX-based systems. The previous article about the check of FreeRDP is available here. ![]() On the other hand, since the projects are pretty small, there aren't many bugs in them anyway :). I will be discussing only those bugs that looked most interesting to me. This is a static analyzer for code written in C, C++, C#, and Java, and it runs on Windows, Linux, and macOS. ![]() The analysis was performed by PVS-Studio. ![]() Today we are going to take a look at the rdesktop client and xrdp server. This is the second post in our series of articles about the results of checking open-source software working with the RDP protocol.
0 Comments
Leave a Reply. |
AuthorWrite something about yourself. No need to be fancy, just an overview. ArchivesCategories |