necouchman commented on pull request #228: URL: https://github.com/apache/guacamole-server/pull/228#issuecomment-693675787
> I think it might be worth considering the guac_argv_await(), guac_argv_register(), etc. family of functions that were recently added as part of GUACAMOLE-221 to make this sort of logic easier Yeah, that makes perfect sense, seeing as it was one of the driving reasons for adding them. Follow-up question, though - if we move to that, is there still any need for the `pthread_cond` and flags? I removed those as part of moving RDP and VNC to these new `argv` functions, but some of the comments you made above make it sounds like it's still worth keeping the flags around and checking that the parameters cannot be manipulated by someone with nefarious intentions? Or does the move to the `argv` handlers also take care of this concern? ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected]
