The front wheel is correct but I would move the D lock at the back. It is only round a wheel and a mudguard.
Your current method means the frame could be stolen by undoing the wheel bolts and lifting away. Can't see exactly how the mudguards are fixed and they might complicate it a bit but not much.
I'd suggest low on the seat tube just above the chain ring and then loop the cable round the front and back wheels and into the D lock.
Edit: forgot. Don't bother looping up through the seat like that. If they want to steal that with the cable there it's only two bolts. It's only one without the cable but if you were still worried you can get security seatpost bolts