В вашем примере ошибок нет. Однородные члены предложения отделяются посредством запятой, тире ставится перед обобщающим словом. Вот правило. Обобщающее слово, стоящее после однородных членов, отделяется от них знаком тире: Поручни, компасы, бинокли, всякие приборы и даже высокие пороги кают **— всё это было медное** (Пауст.); И эти поездки, и наши с ней разговоры **— всё проникнуто было щемящей, безысходной тоской** (Бек.)