Variation in penetrance estimates for BRCA1/2 carriers suggests that other environmental and genetic factors may modify cancer risk in carriers. The GSTM1, T1 and P1 isoenzymes are involved in metabolism of environmental carcinogens. The GSTM1 and GSTT1 gene is absent in a substantial proportion of the population. In GSTP1, a single-nucleotide polymorphism that translates to Ile112Val was associated with lower activity. We studied the effect of these polymorphisms on breast cancer (BC) risk in BRCA1/2 carriers. A population of 320 BRCA1/2 carriers were genotyped; of them 262 were carriers of one of the three Ashkenazi founder mutations. Two hundred and eleven were affected with BC (20 also with ovarian cancer (OC)) and 109 were unaffected with BC (39 of them had OC). Risk analyses were conducted using Cox proportional hazard models adjusted for origin (Ashkenazi vs non-Ashkenazi). We found an estimated BC HR of 0.89 (95% CI 0.65-1.12, P=0.25) and 1.11 (95% CI 0.81-1.52, P=0.53) for the null alleles of GSTM1 and GSTT1, respectively. For GSTP1, HR for BC was 1.36 (95% CI 1.02-1.81, P=0.04) for individuals with Ile/Val, and 2.00 (95% CI 1.18-3.38) for carriers of the Val/Val genotype (P=0.01). An HR of 3.20 (95% CI 1.26-8.09, P=0.01), and younger age at BC onset (P=0.2), were found among Val/Val, BRCA2 carriers, but not among BRCA1 carriers. In conclusion, our results indicate significantly elevated risk for BC in carriers of BRCA2 mutations with GSTP1-Val allele with dosage effect, as implicated by higher risk in homozygous Val carriers. The GSTM1- and GSTT1-null allele did not seem to have a major effect.