Note that in SAP ABAP, sy-subrc == 0 means success. You can use breakpoint before your INSERT statement to ensure if it is a success or not. You can check the possible return value by bringing cursor on it and then press F1 key.
Try checking if/else branch in your code snippet.