There's no harm in having both the active and standby HSRP member router perform the function. The client still only gets one address and, if ever one router is unable to reach the DHCP server (or is itself unreachable) the client continues to get an address without human intervention. I've always set them up just like that with no adverse effect.