Make sure that you don't establish the connection twice by accident (for example in a loop) and that the slot is not using the naming convention of automatic connections:
void on_<widget name>_<signal name>(<signal parameters>);