New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

ballle98/AqualinkD#42: memcpy-param-overlap in PDA Shift processing #87

Merged

sfeakes merged 1 commit into sfeakes:master from ballle98:dev/pda-shift-42

Aug 22, 2019

Contributor

ballle98 commented Aug 22, 2019

fix #86

see see ballle98#42: memcpy-param-overlap in PDA Shift processing


          #42: memcpy-param-overlap in PDA Shift processing

fab8a4d

ballle98 commented

View reviewed changes

pda_menu.c

                     if (getLogLevel() >= LOG_DEBUG){print_menu();}
                   break;
                   case CMD_PDA_SHIFTLINES:
-                    memcpy(_menu[1], _menu[2], (PDA_LINES-1) * (AQ_MSGLEN+1) );
+                    // press up from top - shift menu down by 1

Contributor Author

ballle98 Aug 22, 2019

This is the fix the other code is to test the fic

sfeakes merged commit 118381c into sfeakes:master

ballle98 deleted the dev/pda-shift-42 branch

August 22, 2019 13:43

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet