Мне не совсем понятна логика вашего кода зачем там вообще нужен цикл
$('.red').click(function(){
$(this).parent('.colors').siblings('.product_circle').find('img').attr('src', 'img/bag_1_red.png');
});
если вы хотите применить событие ко всем элиментам с класом red то для этого не нужен цикл просто у элиментов должен быть одинаковый клас остальное зделает jquery