If there is no reason that your shield MUST be on top, adding full headers for all pins, including the ICSP pins would be a good idea.
Reaching the reset pin on the Arduino, under your shield, could be a problem. Adding a reset pin on your shield would be a good idea, too.