Whether it's right or not depends on what it is you want it to do!
But after having a quick look, these things strike me as odd:
1) The cluster control on the front panel of your main VI gets unbundled, but the data doesn't go anywhere.
2) The sub-VI outputs a boolean, but nothing is done with this output in the main program.