Plot hole: It is apparent that Shrek and Donkey have never traveled to the land of Far Far Away, yet Donkey immediately recognizes the "old Keeblers' place" when he should have no idea where it is or what it looks like.
Suggested correction: Donkey only thinks it's the "old Keebler's place." Puss in Boots then corrects him and says it's the Fairy Godmother's cottage.
Plot hole: In the scene where the Sheriff stops the cows' brawl, he gives them to the Chinese man. The Sheriff knows what Pearl's animals look like, so why didn't he recognize the cows as Pearl's?