﻿<?xml version="1.0" encoding="utf-8"?>
<InstanceReport xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xmlns:xsd="http://www.w3.org/2001/XMLSchema">
  <Version>2.2.0.7</Version>
  <hasSegments>false</hasSegments>
  <ReportName>Commitments and Contingencies</ReportName>
  <ReportLongName>0203 - Disclosure - Commitments and Contingencies</ReportLongName>
  <DisplayLabelColumn>true</DisplayLabelColumn>
  <ShowElementNames>false</ShowElementNames>
  <RoundingOption />
  <HasEmbeddedReports>false</HasEmbeddedReports>
  <Columns>
    <Column>
      <LabelColumn>false</LabelColumn>
      <Id>1</Id>
      <Labels>
        <Label Id="1" Label="6 Months Ended" />
        <Label Id="2" Label="Jun. 30, 2010" />
      </Labels>
      <CurrencyCode>USD</CurrencyCode>
      <FootnoteIndexer />
      <hasSegments>false</hasSegments>
      <hasScenarios>false</hasScenarios>
      <Segments />
      <Scenarios />
      <Units>
        <Unit>
          <UnitID>USD</UnitID>
          <UnitType>Standard</UnitType>
          <StandardMeasure>
            <MeasureSchema>http://www.xbrl.org/2003/iso4217</MeasureSchema>
            <MeasureValue>USD</MeasureValue>
            <MeasureNamespace>iso4217</MeasureNamespace>
          </StandardMeasure>
          <Scale>0</Scale>
        </Unit>
        <Unit>
          <UnitID>USDEPS</UnitID>
          <UnitType>Divide</UnitType>
          <NumeratorMeasure>
            <MeasureSchema>http://www.xbrl.org/2003/iso4217</MeasureSchema>
            <MeasureValue>USD</MeasureValue>
            <MeasureNamespace>iso4217</MeasureNamespace>
          </NumeratorMeasure>
          <DenominatorMeasure>
            <MeasureSchema>http://www.xbrl.org/2003/instance</MeasureSchema>
            <MeasureValue>shares</MeasureValue>
            <MeasureNamespace>xbrli</MeasureNamespace>
          </DenominatorMeasure>
          <Scale>0</Scale>
        </Unit>
        <Unit>
          <UnitID>Shares</UnitID>
          <UnitType>Standard</UnitType>
          <StandardMeasure>
            <MeasureSchema>http://www.xbrl.org/2003/instance</MeasureSchema>
            <MeasureValue>shares</MeasureValue>
            <MeasureNamespace>xbrli</MeasureNamespace>
          </StandardMeasure>
          <Scale>0</Scale>
        </Unit>
      </Units>
      <CurrencySymbol>$</CurrencySymbol>
    </Column>
  </Columns>
  <Rows>
    <Row>
      <Id>2</Id>
      <Label>Commitments and Contingencies [Abstract]</Label>
      <Level>0</Level>
      <ElementName>bpl_CommitmentsAndContingenciesAbstract</ElementName>
      <ElementPrefix>bpl</ElementPrefix>
      <IsBaseElement>false</IsBaseElement>
      <BalanceType>na</BalanceType>
      <PeriodType>duration</PeriodType>
      <ShortDefinition>Commitments And Contingencies.</ShortDefinition>
      <IsReportTitle>false</IsReportTitle>
      <IsSegmentTitle>false</IsSegmentTitle>
      <IsSubReportEnd>false</IsSubReportEnd>
      <IsCalendarTitle>false</IsCalendarTitle>
      <IsTuple>false</IsTuple>
      <IsAbstractGroupTitle>true</IsAbstractGroupTitle>
      <IsEquityPrevioslyReportedAsRow>false</IsEquityPrevioslyReportedAsRow>
      <IsEquityAdjustmentRow>false</IsEquityAdjustmentRow>
      <IsBeginningBalance>false</IsBeginningBalance>
      <IsEndingBalance>false</IsEndingBalance>
      <IsReverseSign>false</IsReverseSign>
      <PreferredLabelRole />
      <IsEPS>false</IsEPS>
      <FootnoteIndexer />
      <Cells>
        <Cell>
          <Id>1</Id>
          <ShowCurrencySymbol>false</ShowCurrencySymbol>
          <IsNumeric>false</IsNumeric>
          <IsRatio>false</IsRatio>
          <DisplayZeroAsNone>false</DisplayZeroAsNone>
          <NumericAmount>0</NumericAmount>
          <RoundedNumericAmount>0</RoundedNumericAmount>
          <NonNumbericText />
          <NonNumericTextHeader />
          <FootnoteIndexer />
          <hasSegments>false</hasSegments>
          <hasScenarios>false</hasScenarios>
          <DisplayDateInUSFormat>false</DisplayDateInUSFormat>
        </Cell>
      </Cells>
      <OriginalInstanceReportColumns />
      <ElementDataType>xbrli:stringItemType</ElementDataType>
      <SimpleDataType>string</SimpleDataType>
      <ElementDefenition>Commitments And Contingencies.</ElementDefenition>
      <IsTotalLabel>false</IsTotalLabel>
    </Row>
    <Row>
      <Id>3</Id>
      <Label>COMMITMENTS AND CONTINGENCIES</Label>
      <Level>1</Level>
      <ElementName>us-gaap_CommitmentsAndContingenciesDisclosureTextBlock</ElementName>
      <ElementPrefix>us-gaap</ElementPrefix>
      <IsBaseElement>true</IsBaseElement>
      <BalanceType>na</BalanceType>
      <PeriodType>duration</PeriodType>
      <ShortDefinition>No definition available.</ShortDefinition>
      <IsReportTitle>false</IsReportTitle>
      <IsSegmentTitle>false</IsSegmentTitle>
      <IsSubReportEnd>false</IsSubReportEnd>
      <IsCalendarTitle>false</IsCalendarTitle>
      <IsTuple>false</IsTuple>
      <IsAbstractGroupTitle>false</IsAbstractGroupTitle>
      <IsEquityPrevioslyReportedAsRow>false</IsEquityPrevioslyReportedAsRow>
      <IsEquityAdjustmentRow>false</IsEquityAdjustmentRow>
      <IsBeginningBalance>false</IsBeginningBalance>
      <IsEndingBalance>false</IsEndingBalance>
      <IsReverseSign>false</IsReverseSign>
      <PreferredLabelRole>verboselabel</PreferredLabelRole>
      <IsEPS>false</IsEPS>
      <FootnoteIndexer />
      <Cells>
        <Cell>
          <Id>1</Id>
          <ShowCurrencySymbol>false</ShowCurrencySymbol>
          <IsNumeric>false</IsNumeric>
          <IsRatio>false</IsRatio>
          <DisplayZeroAsNone>false</DisplayZeroAsNone>
          <NumericAmount>0</NumericAmount>
          <RoundedNumericAmount>0</RoundedNumericAmount>
          <NonNumbericText>&lt;!--DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd" --&gt;
   &lt;!-- Begin Block Tagged Note 3 - us-gaap:CommitmentsAndContingenciesDisclosureTextBlock--&gt;
   &lt;div style="font-family: 'Times New Roman',Times,serif"&gt;
   &lt;div align="left" style="font-size: 10pt; margin-top: 12pt"&gt;&lt;b&gt;3. COMMITMENTS AND CONTINGENCIES&lt;/b&gt;
   &lt;/div&gt;
   &lt;div align="left" style="font-size: 10pt; margin-top: 12pt"&gt;&amp;#160;&amp;#160;&lt;i&gt;Claims and Proceedings&lt;/i&gt;
   &lt;/div&gt;
   &lt;div align="left" style="font-size: 10pt; margin-top: 6pt"&gt;&amp;#160;&amp;#160;&amp;#160;&amp;#160;&amp;#160;In the ordinary course of business, we are involved in various claims and legal proceedings,
   some of which are covered by insurance. We are generally unable to predict the timing or outcome of
   these claims and proceedings. Based upon our evaluation of existing claims and proceedings and the
   probability of losses relating to such contingencies, we have accrued certain amounts relating to
   such claims and proceedings, none of which are considered material.
   &lt;/div&gt;
   &lt;div align="left" style="font-size: 10pt; margin-top: 6pt"&gt;&amp;#160;&amp;#160;&amp;#160;&amp;#160;&amp;#160;In April&amp;#160;2010, the Pipeline Hazardous Materials Safety Administration (&amp;#8220;PHMSA&amp;#8221;) proposed
   penalties totaling approximately $0.5&amp;#160;million in connection with a tank overfill incident that
   occurred at our facility in East Chicago, Indiana, in May&amp;#160;2005 and other related personnel
   qualification issues raised as a result of PHMSA&amp;#8217;s 2008 Integrity Inspection. We are contesting the
   proposed penalty. The timing or outcome of this appeal cannot reasonably be determined at this
   time.
   &lt;/div&gt;
   &lt;div align="left" style="font-size: 10pt; margin-top: 6pt"&gt;&amp;#160;&amp;#160;&amp;#160;&amp;#160;&amp;#160;On July
   30, 2010, a putative class action was filed by a unitholder against BGH, MainLine Management
   LLC (&amp;#8220;MainLine Management&amp;#8221;), BGH GP Holdings, LLC (&amp;#8220;BGH GP&amp;#8221;), and each of MainLine Management&amp;#8217;s
   directors in the District Court of Harris County, Texas under the
   caption &lt;i&gt;Broadbased
   Equities v. Forrest E. Wylie, et. al.&lt;/i&gt;  In the Petition, the plaintiff alleges that MainLine
   Management and its directors breached their fiduciary duties to BGH&amp;#8217;s public unitholders by,
   among other things, acting to facilitate the sale of BGH to Buckeye in order to facilitate
   the gradual sale by BGH GP of its interest in BGH and failing to disclose all material facts
   in order that the BGH unitholders can cast an informed vote on the Merger Agreement.  Among
   other things, the Petition seeks an order certifying a class consisting of all BGH
   unitholders, a determination that the action is a proper derivative action, damages
   in an unspecified amount, and an award of attorneys&amp;#8217; fees and costs.  The defendants
   have not yet answered or otherwise responded to the Petition.
   &lt;/div&gt;
   &lt;div align="left" style="font-size: 10pt; margin-top: 6pt"&gt;&amp;#160;&amp;#160;&amp;#160;&amp;#160;&amp;#160;On August
   2, 2010, a putative class action was filed by a unitholder against BGH, MainLine Management,
   Grand Ohio, LLC, Buckeye, Buckeye GP, and each of MainLine Management&amp;#8217;s directors in the
   District Court of Harris County, Texas under the caption &lt;i&gt;Henry James Steward v. Forrest
   E. Wylie, et. al.&lt;/i&gt;  In the Petition, the plaintiff alleges that MainLine Management and
   its directors breached their fiduciary duties to BGH&amp;#8217;s public unitholders by, among other
   things, failing to disclose all material facts in order that the BGH unitholders can cast
   an informed vote on the Merger Agreement.  The Petition also alleges that Buckeye,
   Buckeye GP and Grand Ohio, LLC aided and abetted the breaches of fiduciary duty.  Among
   other things, the Petition seeks an order certifying a plaintiff class consisting of
   all of BGH unitholders, an order enjoining the Merger, rescission of the Merger, damages
   in an unspecified amount, and an award of attorneys&amp;#8217; fees and costs.  Neither we nor the
   other defendants have yet answered or otherwise responded to the Petition.
   &lt;/div&gt;
   &lt;div align="left" style="font-size: 10pt; margin-top: 6pt"&gt;&amp;#160;&amp;#160;&amp;#160;&amp;#160;&amp;#160;On August
   2, 2010, a putative class action was filed by a unitholder against BGH, MainLine Management,
   BGH GP, ArcLight Capital Partners, LLC (&amp;#8220;ArcLight&amp;#8221;), Kelso &amp;#038; Company (&amp;#8220;Kelso&amp;#8221;), Buckeye,
   Buckeye GP, and each of MainLine Management&amp;#8217;s directors, in the District Court of Harris
   County, Texas under the caption &lt;i&gt;Henry James Steward v. Forrest E. Wylie, et. al.&lt;/i&gt;  In the
   Petition, the plaintiff alleges that MainLine Management and its directors breached their
   fiduciary duties to BGH&amp;#8217;s public unitholders by, among other things, accepting insufficient
   consideration, failing to condition the Merger on a majority vote of public unitholders of
   BGH, and failing to disclose all material facts in order that the BGH unitholders can cast
   an informed vote on the Merger Agreement.  The Petition also alleges that Buckeye, Buckeye
   GP, BGH GP, ArcLight, and Kelso aided and abetted the breaches of fiduciary duty.  Among
   other things, the Petition seeks an order certifying a class consisting of all of BGH&amp;#8217;s
   unitholders, an order enjoining the Merger, damages in an unspecified amount, and an award
   of attorneys&amp;#8217; fees and costs.  Neither we nor the other defendants have yet answered or
   otherwise responded to the Petition.
   &lt;/div&gt;
   &lt;div align="left" style="font-size: 10pt; margin-top: 12pt"&gt;&amp;#160;&amp;#160;&lt;i&gt;Environmental Contingencies&lt;/i&gt;
   &lt;/div&gt;
   &lt;div align="left" style="font-size: 10pt; margin-top: 6pt"&gt;&amp;#160;&amp;#160;&amp;#160;&amp;#160;&amp;#160;In accordance with our accounting policy, we recorded operating expenses, net of insurance
   recoveries, of $2.5&amp;#160;million and $1.2&amp;#160;million during the three months ended June&amp;#160;30, 2010 and 2009,
   respectively, and $5.4&amp;#160;million and $6.6&amp;#160;million during the six months ended June&amp;#160;30, 2010 and 2009,
   respectively, related to environmental expenditures unrelated to claims and proceedings.
   &lt;/div&gt;
   &lt;div align="left" style="font-size: 10pt; margin-top: 12pt"&gt;&amp;#160;&amp;#160;&lt;i&gt;Ammonia Contract Contingencies&lt;/i&gt;
   &lt;/div&gt;
   &lt;div align="left" style="font-size: 10pt; margin-top: 6pt"&gt;&amp;#160;&amp;#160;&amp;#160;&amp;#160;&amp;#160;On November&amp;#160;30, 2005, Buckeye Gulf Coast Pipe Lines, L.P. (&amp;#8220;BGC&amp;#8221;) purchased an ammonia
   pipeline and other assets from El Paso Merchant Energy-Petroleum Company (&amp;#8220;EPME&amp;#8221;), a subsidiary of
   El Paso Corporation (&amp;#8220;El Paso&amp;#8221;). As part of the transaction, BGC assumed the obligations of EPME
   under several contracts involving monthly purchases and sales of ammonia. EPME and BGC agreed,
   however, that EPME would retain the economic risks and benefits associated with those contracts
   until their expiration at the end of 2012. To effectuate this agreement, BGC passes through to
   EPME both the cost of purchasing ammonia under a supply contract and the proceeds from selling
   ammonia under three sales contracts. For the vast majority of monthly periods since the closing of
   the pipeline acquisition, the pricing terms of the ammonia contracts have resulted in ammonia costs
   exceeding ammonia sales proceeds. The amount of the shortfall generally increases as the market
   price of ammonia increases.
   &lt;/div&gt;
   &lt;div align="left" style="font-size: 10pt; margin-top: 6pt"&gt;&amp;#160;&amp;#160;&amp;#160;&amp;#160;&amp;#160;EPME has informed BGC that, notwithstanding the parties&amp;#8217; agreement, it will not continue to
   pay BGC for shortfalls created by the pass-through of ammonia costs in excess of ammonia revenues.
   EPME encouraged BGC to seek payment by invoking a $40.0&amp;#160;million guaranty made by El Paso, which
   guaranteed EPME&amp;#8217;s obligations to BGC. If EPME fails to reimburse BGC for these shortfalls for a
   significant period during the remainder of the term of the ammonia agreements, then such
   unreimbursed shortfalls could exceed the $40.0&amp;#160;million cap on El Paso&amp;#8217;s guaranty. To the extent
   the unreimbursed shortfalls significantly exceed the $40.0&amp;#160;million cap, the resulting costs
   incurred by BGC could adversely affect our financial position, results of operations and cash
   flows. To date, BGC has continued to receive payment for ammonia costs under the contracts at
   issue. BGC has not called on El Paso&amp;#8217;s guaranty and believes only BGC may invoke the guaranty.
   EPME, however, contends that El Paso&amp;#8217;s guaranty is the source of payment for the shortfalls, but
   has not clarified the extent to which it believes the guaranty has been exhausted. We have been
   working with EPME to terminate the ammonia sales contracts and ammonia supply contracts and, at no
   out of pocket cost to us, have terminated one of the ammonia sales contracts. Given, however, the
   uncertainty of future ammonia prices and EPME&amp;#8217;s future actions, we continue to believe we have risk
   of loss and, at this time, are unable to estimate the amount of any such losses we might incur in
   the future. We are assessing our options in the event that we and EPME are unable to terminate the
   remaining contracts or otherwise mitigate the remaining risk, including potential recourse against
   EPME and El Paso, with respect to this matter.
   &lt;/div&gt;
   &lt;!-- Folio --&gt;
   &lt;!-- /Folio --&gt;
   &lt;/div&gt;
   &lt;!-- PAGEBREAK --&gt;
   &lt;div style="font-family: 'Times New Roman',Times,serif"&gt;
   &lt;div align="center" style="font-size: 10pt; margin-top: 0pt"&gt;
   &lt;b&gt;
   &lt;/b&gt;
   &lt;/div&gt;
   &lt;div align="left" style="font-size: 10pt; margin-top: 6pt"&gt;&amp;#160;&amp;#160;&lt;i&gt;Customer Bankruptcy&lt;/i&gt;
   &lt;/div&gt;
   &lt;div align="left" style="font-size: 10pt; margin-top: 6pt"&gt;&amp;#160;&amp;#160;&amp;#160;&amp;#160;&amp;#160;One of our customers filed for bankruptcy in October&amp;#160;2009; approximately $4.2&amp;#160;million remained
   payable to us from the customer pursuant to a pre-bankruptcy contract. In June&amp;#160;2010, we entered
   into a settlement with the bankrupt customer and its largest creditor pursuant to which we expect
   to be paid at least $2.0&amp;#160;million upon the sale of certain of the customer&amp;#8217;s assets within the
   bankruptcy proceedings, and we were released from both asserted and unasserted claims. At this
   time, we expect the sale of the assets to be completed, and the settlement payment to be made to
   us, in the third quarter of 2010. As a result of the settlement, our Development &amp;#038; Logistics
   segment recognized approximately $2.1&amp;#160;million in expense related to the write-off of a portion of
   the outstanding receivable balance during the three and six months ended June&amp;#160;30, 2010.
   &lt;/div&gt;
   &lt;div align="left"&gt;
   &lt;/div&gt;
   &lt;/div&gt;
</NonNumbericText>
          <NonNumericTextHeader>&lt;!--DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd" --&gt;
   &lt;!-- Begin Block Tagged Note</NonNumericTextHeader>
          <FootnoteIndexer />
          <hasSegments>false</hasSegments>
          <hasScenarios>false</hasScenarios>
          <DisplayDateInUSFormat>false</DisplayDateInUSFormat>
        </Cell>
      </Cells>
      <OriginalInstanceReportColumns />
      <ElementDataType>us-types:textBlockItemType</ElementDataType>
      <SimpleDataType>textblock</SimpleDataType>
      <ElementDefenition>Includes disclosure of commitments and contingencies. This element may be used as a single block of text to encapsulate the entire disclosure including data and tables.</ElementDefenition>
      <ElementReferences>Reference 1: http://www.xbrl.org/2003/role/presentationRef
 -Publisher FASB
 -Name FASB Interpretation (FIN)
 -Number 14
 -Paragraph 3

Reference 2: http://www.xbrl.org/2003/role/presentationRef
 -Publisher FASB
 -Name Statement of Financial Accounting Standard (FAS)
 -Number 5
 -Paragraph 9, 10, 11, 12

</ElementReferences>
      <IsTotalLabel>false</IsTotalLabel>
    </Row>
  </Rows>
  <Footnotes />
  <NumberOfCols>1</NumberOfCols>
  <NumberOfRows>2</NumberOfRows>
  <HasScenarios>false</HasScenarios>
  <MonetaryRoundingLevel>UnKnown</MonetaryRoundingLevel>
  <SharesRoundingLevel>UnKnown</SharesRoundingLevel>
  <PerShareRoundingLevel>UnKnown</PerShareRoundingLevel>
  <HasPureData>false</HasPureData>
  <SharesShouldBeRounded>true</SharesShouldBeRounded>
</InstanceReport>
